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

Jump to bottom

rename classes #21

Closed
InfectedBytes opened this issue May 1, 2018 · 1 comment
Closed

rename classes #21

InfectedBytes opened this issue May 1, 2018 · 1 comment
Milestone
v2.4.0 - BigRefactor

Comments

@InfectedBytes
Copy link
Owner

At least one Manager class should be renamed, because it confuses users that there are two different classes with the same name.
@InfectedBytes
Copy link
Owner Author

I should probably go with something like GameJoltAPI and GameJoltUI.
Yes, it looks a bit strange and redundant if you use the fully qualified name (GameJolt.API.GameJoltAPI), but who uses the fully qualified name? exactly, no one.

@InfectedBytes InfectedBytes added this to the 2.4 milestone May 1, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.4.0 - BigRefactor
Development

No branches or pull requests
1 participant
@InfectedBytes