Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

All namespaces should be titlecase #278

Closed
nrkramer opened this issue May 6, 2023 · 3 comments
Closed

All namespaces should be titlecase #278

nrkramer opened this issue May 6, 2023 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers Priority:Low Priority Label for low priority issue

Comments

@nrkramer
Copy link
Collaborator

nrkramer commented May 6, 2023

The Namespace for the library should match the name you've got here on Github
@nrkramer
Copy link
Collaborator Author

nrkramer commented May 6, 2023

Additionally, all the other namespaces should use title-case as well (such as PARTITIONING -> Partitioning)

@ZigRazor
Copy link
Owner

ZigRazor commented May 7, 2023

Yes, I agree with your observation!

@ZigRazor ZigRazor added enhancement New feature or request good first issue Good for newcomers Priority:Low Priority Label for low priority issue labels May 7, 2023
@nrkramer nrkramer changed the title Change namespace from CXXGRAPH -> CXXGraph? All namespaces should be titlecase May 7, 2023
@nrkramer nrkramer mentioned this issue May 7, 2023
Merged
nrkramer added a commit that referenced this issue May 8, 2023
@nrkramer
(Issue #278) All namespaces should be titlecase
d95f9a4
@nrkramer
Copy link
Collaborator Author

nrkramer commented May 8, 2023

This is done with #281

@nrkramer nrkramer closed this as completed May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers Priority:Low Priority Label for low priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nrkramer @ZigRazor