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

Update Rubocop version and enforce all cops #617

Merged
merged 5 commits into from
Apr 17, 2024
Merged

Conversation

Splines
Copy link
Member

@Splines Splines commented Apr 15, 2024

Commands used to run Rubocop

bundle exec rubocop --autocorrect --disable-uncorrectable
bundle exec rubocop -A  # in a second run

For reviewers

  • You have to run bundle install locally to install the new versions.
  • Please review the unsafe cops carefully as they might introduce breaking behavior. E.g. the autocorrection of Style/MapIntoArray failed for us because we return an object and not an array. I fixed that, also see the commit body of afccc70.
  • Note that I disabled one potential fix in a migration file as we don't want to alter db migration files once they've already been deployed to production.

One rule was disabled since we don't want to alter already performed
migrations.
Special care is needed for `tag.rb` around line 190.
The autofix did not produce a valid result, I had to put extra
curly braces around that line as we want to push an object to the array.
@Splines Splines added the dependencies Pull requests that update a dependency file label Apr 15, 2024
@Splines Splines self-assigned this Apr 15, 2024
@Splines Splines marked this pull request as ready for review April 15, 2024 21:20
fosterfarrell9
fosterfarrell9 previously approved these changes Apr 16, 2024
They probably got unnecessary due to the update of RuboCop, though I
haven't looked into their changelog to confirm.
@Splines Splines merged commit dfdad5d into dev Apr 17, 2024
3 checks passed
@Splines Splines deleted the deps/update-rubocop branch April 17, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants