Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

Update to a version of Rubocop that is compatible with Ruby 2.4-3.1 #525

Merged
merged 3 commits into from
Apr 8, 2022
Merged

Update to a version of Rubocop that is compatible with Ruby 2.4-3.1 #525

merged 3 commits into from
Apr 8, 2022

Conversation

petergoldstein
Copy link
Contributor

This is an update to Rubocop that maintains compatibility with Ruby 2.4-3.1 using a common .rubocop_rspec_base.yml file across rspec-core, rspec-expectations, rspec-mocks, and rspec-support

Changes to this common base configuration file are minimal, limited to:

  1. Adding missing Departments for Cops
  2. Splitting/renaming Cop configurations where directed by Rubocop for pre-existing configurations.
  3. Removing a Performance cop configuration as that is now packaged in a separate library.

Each repo now has a .rubocop_todo.yml file which can be worked through at the maintainers leisure.

Some updates to the existing .rubocop.yml files were made, mostly in removing duplication and updating some variables so Rubocop would run green.

I also added some respond_to? guards for two undef statements.

@@ -1,79 +1,76 @@
# This file was generated on 2021-07-15T10:45:53+01:00 from the rspec-dev repo.
# This file was generated on 2021-07-15T10:45:52+01:00 from the rspec-dev repo.
Copy link
Member

Choose a reason for hiding this comment

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

We generally update this file using rspec-dev that can make multiple changes at once, I'd prefer to do that and then add the extra "version bump" commit to those prs by hand.

@pirj pirj merged commit 1811c9c into rspec:main Apr 8, 2022
@petergoldstein petergoldstein deleted the feature/rubocop_update branch June 12, 2022 19:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants