Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor/ unify codebase naming #121

Merged
merged 5 commits into from
Aug 3, 2024

Conversation

simon-the-shark
Copy link
Member

Re-pr of #119

Unified names refactor with simple guidelines and note in readme. You can take a quick look, but I wouldn't spend too much time looking into this, this should be semantically the same, just unified names. We can also obviously discuss some naming choices. Tell me what you think

@simon-the-shark simon-the-shark self-assigned this Aug 3, 2024
@simon-the-shark simon-the-shark force-pushed the refactor/navigation-remake branch from eece632 to cfe9917 Compare August 3, 2024 12:41
@simon-the-shark simon-the-shark changed the title refactor: unife codebase naming Refactor/ unify codebase naming Aug 3, 2024
Copy link
Member

@mikolaj-jalocha mikolaj-jalocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@simon-the-shark simon-the-shark merged commit a2ec323 into refactor/navigation-remake Aug 3, 2024
2 checks passed
@simon-the-shark simon-the-shark deleted the refactor/unify-names branch August 3, 2024 20:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants