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.

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

refactor: group var together #18

Closed
wants to merge 1 commit into from
Closed

refactor: group var together #18

wants to merge 1 commit into from

Conversation

SeaRoll
Copy link
Contributor

@SeaRoll SeaRoll commented Oct 25, 2024

just a suggestion but I wonder if this looks more cleaner since it's better to have global variables in an application defined together instead of different places 🤔

@raeperd
Copy link
Owner

raeperd commented Oct 25, 2024

I choose conciseness.
Each global variable is located as close as function that uses value.
And those globals are not intended to be used else where.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants