Skip to content

Pluginfy RuboCop Committee#8

Merged
ydah merged 1 commit intomainfrom
pluginfy_with_lint_roller
Apr 2, 2025
Merged

Pluginfy RuboCop Committee#8
ydah merged 1 commit intomainfrom
pluginfy_with_lint_roller

Conversation

@ydah
Copy link
Owner

@ydah ydah commented Apr 2, 2025

This PR adds support for RuboCop's Plugin feature.

It replaces the ad-hoc Inject with RuboCop plugins introduced in RuboCop 1.72.

NOTE: Version 1.72.1 or later is specified, as it includes rubocop/rubocop#13837, which removes the Inject from spec_helper.rb. While 1.72.0 is sufficient for runtime, but specifying the patch version 1.72.1 keeps things simple.


Before submitting the PR make sure the following are checked:

  • Feature branch is up-to-date with master (if not - rebase it).
  • Wrote [good commit messages][1].
  • [-] Commit message starts with [Fix #issue-number] (if the related issue exists).
  • Squashed related commits together.
  • Added tests.
  • Updated documentation.
  • Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
  • The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@ydah ydah force-pushed the pluginfy_with_lint_roller branch from e2f4d8e to 6e91d52 Compare April 2, 2025 13:54
This PR adds support for RuboCop's Plugin feature.

It replaces the ad-hoc Inject with RuboCop plugins introduced in RuboCop 1.72.

NOTE: Version 1.72.1 or later is specified, as it includes rubocop/rubocop#13837, which removes the Inject from spec_helper.rb. While 1.72.0 is sufficient for runtime, but specifying the patch version 1.72.1 keeps things simple.
@ydah ydah force-pushed the pluginfy_with_lint_roller branch from 6e91d52 to b099d30 Compare April 2, 2025 14:00
@ydah ydah merged commit 452469c into main Apr 2, 2025
35 of 36 checks passed
@ydah ydah deleted the pluginfy_with_lint_roller branch April 2, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant