platanus / potassium

A Rails application generator by Platanus, inspired by Suspenders
MIT License
232 stars 17 forks source link

feat(rubocop): correct all rubocop erros #390

Open Baelfire18 opened 3 years ago

Baelfire18 commented 3 years ago

Hi I was iniciating a proyecting using this awsome generator and i really it came with some default robocop errors, so I wanted to contribute and help correcting them. Can you review my PR?

What?

Why?

How?

rjherrera commented 3 years ago

@Baelfire18 Hi there! Thanks for the pull request and sorry for the late reply.

We talked about this change internally and we appreciate the PR, however, we don't want to change those rules as they are the rules we use in many projects and that we are used to.

Regardless of that, we also saw that many changes that this PR introduces comply with the original set of rubocop rules, but were not fixed until now. If you want you could change this PR to only fix errors that applied before the rule change.

We could then merge those changes (the ones consistent with the original rules) but we will not change the set of rubocop rules in this project for the foreseeable future.

Baelfire18 commented 3 years ago

Okay, totally understandable. Should I revert this commit to @rjherrera ?