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 #364

Merged
merged 2 commits into from
Jan 4, 2022
Merged

Update rubocop #364

merged 2 commits into from
Jan 4, 2022

Conversation

olbrich
Copy link
Contributor

@olbrich olbrich commented Dec 29, 2021

  • Removes version constraint on Rubocop
  • Add rubocop-minitest

I deliberately did not apply any rubocop auto corrections in this branch. There are a lot of them and not all of them are correct or desirable, so we should do those in smaller batches.

@olbrich olbrich self-assigned this Dec 29, 2021
@olbrich olbrich added this to the 0.18.0 milestone Dec 29, 2021
@coveralls
Copy link
Collaborator

coveralls commented Dec 29, 2021

Coverage Status

Coverage remained the same at 97.927% when pulling 363a0d5 on olbrich:update-rubocop into 550bbb6 on devise-security:master.

@olbrich olbrich added ⬆️ dependencies Pull requests that update a dependency file 🚧 WIP Work in progress labels Dec 29, 2021
@@ -1,5 +1,6 @@
require:
- rubocop-rails
- rubocop-minitest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What stuff does this bring in?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds some additional Cops for minitest-specific things.

@olbrich olbrich marked this pull request as ready for review December 29, 2021 22:07
@olbrich olbrich removed the 🚧 WIP Work in progress label Dec 29, 2021
@olbrich olbrich merged commit e7abd25 into devise-security:master Jan 4, 2022
@olbrich olbrich deleted the update-rubocop branch January 4, 2022 16:18
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.

None yet

3 participants