AegisNimbus / PuckMaster

MIT License
4 stars 4 forks source link

Refactor Module-Level Constants to Enums and Classes #1

Open ShearStress021 opened 1 week ago

ShearStress021 commented 1 week ago

Screenshot 2024-10-12 151352

The current codebase utilizes module-level constants for defining various settings, configurations, and related values. While this approach is straightforward, it can lead to issues with organization, readability, and type safety as the project grows.

github-actions[bot] commented 1 week ago

Thank you for creating this issue! 🎉 @SUGAM-ARORA @OJAS-ARORA will look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊

You can also check our CONTRIBUTE.md for guidelines on contributing to this project.

Ojas-Arora commented 6 days ago

@ShearStress021 Tell the solution that you will provide for this.

ShearStress021 commented 6 days ago

i would cluster related constants together with either enum, class or namedtuples considering the relationship and datatype of the constants

Ojas-Arora commented 6 days ago

@ShearStress021 Ok, Issue assigned to you.