Solidus: Rename one letter variables

Created on 3 Aug 2019  路  3Comments  路  Source: solidusio/solidus

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.

Beginner-Friendly Hacktoberfest

Most helpful comment

Thanks to @mkoltonski, @JDutil and @juliannatetreault for the work on this issue!

All 3 comments

Hi
I can handle it if you don't mind

Hey @mkoltonski I would love to have your help with this. Here is a good trick to find these variables:

# in the solidus repository
gem install reek
reek --smell UncommunicativeVariableName

That should give you a list of them 馃憤

Thanks to @mkoltonski, @JDutil and @juliannatetreault for the work on this issue!

Was this page helpful?
0 / 5 - 0 ratings