-
Notifications
You must be signed in to change notification settings - Fork 29
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.
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
Split up CosmeticColors.py, and organize #2263
Comments
Imo anything above 1k indicates the file should be broken up or refactored. |
Yea, got a few good splits for this |
Current progress: CosmeticColors.py is now down to under 2k lines. Still a lot of work to do (Most of that 2k lines is enemy recolors, which I seriously need to refactor to make less of a pain to look at) Also, I've noticed that the colorblind section needs a serious refactor and rewrite to make it less convoluted. I think this was one of Seagull's first projects, so this makes sense. The colorblind file is sitting at a little over 1k lines, and I think we can get it to under 700 with a bit of refactor |
4k+ line python file with many functions hurts me
The text was updated successfully, but these errors were encountered: