Skip to content

Fixes Rubocop require/plugin warning#6783

Merged
compwron merged 1 commit intorubyforgood:mainfrom
hexdevs:sb-fix-rubocop-warning-6782
Mar 20, 2026
Merged

Fixes Rubocop require/plugin warning#6783
compwron merged 1 commit intorubyforgood:mainfrom
hexdevs:sb-fix-rubocop-warning-6782

Conversation

@stefannibrasil
Copy link
Copy Markdown
Contributor

@stefannibrasil stefannibrasil commented Mar 16, 2026

What github issue is this PR for, if any?

Resolves #6782

What changed, and why?

When running rubocop, this warning is printed:

 rubocop-factory_bot extension supports plugin, specify `plugins: rubocop-factory_bot` instead of `require: rubocop-factory_bot` in /home/runner/work/casa/casa/.rubocop.yml.
For more information, see https://docs.rubocop.org/rubocop/plugin_migration_guide.html.

To fix it, we need to add that library as a plugin: https://docs.rubocop.org/rubocop/latest/plugin_migration_guide.html#for-plugin-users

How is this tested?

CI before: https://github.com/rubyforgood/casa/actions/runs/23155851553/job/67270244414

CI after: https://github.com/rubyforgood/casa/actions/runs/23156788730/job/67273625209

When running rubocop, this warning is printed:

 rubocop-factory_bot extension supports plugin, specify `plugins: rubocop-factory_bot` instead of `require: rubocop-factory_bot` in /home/runner/work/casa/casa/.rubocop.yml.
For more information, see https://docs.rubocop.org/rubocop/plugin_migration_guide.html.

To fix it, we need to add that library as a plugin: https://docs.rubocop.org/rubocop/latest/plugin_migration_guide.html#for-plugin-users
Copy link
Copy Markdown
Collaborator

@compwron compwron left a comment

Choose a reason for hiding this comment

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

Great little fix, thanks. :) This has been frustrating me

@compwron compwron merged commit 06a54d7 into rubyforgood:main Mar 20, 2026
11 checks passed
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.

Rubocop deprecation warning

2 participants