-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Rename one letter variables #3294
Comments
Hi |
Hey @mkoltonski I would love to have your help with this. Here is a good trick to find these variables:
That should give you a list of them 👍 |
4 tasks
juliannatetreault
added a commit
to juliannatetreault/solidus
that referenced
this issue
Oct 24, 2019
5 tasks
5 tasks
juliannatetreault
added a commit
to juliannatetreault/solidus
that referenced
this issue
Oct 24, 2019
Thanks to @mkoltonski, @JDutil and @juliannatetreault for the work on this issue! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Throughout Solidus' codebase there are many one-letter variables. These are not ideal as they are not expressive and meaningful for human beings.
I think these should be replaced with human intelligible words, as @jacobherrington already did with some in #3292.
This is a beginner-friendly issue, especially suited for anybody interested in starting helping with open source.
The text was updated successfully, but these errors were encountered: