Cleanup some more residual `test-prof` workarounds by Rylan12 · Pull Request #21643 · Homebrew/brew
Pull request overview
Removes leftover test-prof-specific RuboCop loading workarounds now that the RuboCop plugin/bundler setup is expected to work without manual requires.
Changes:
- Drop the explicit
require "rubocop/test_prof"from the RuboCop wrapper entrypoint. - Simplify the RuboCop integration spec to run
rubocopwithout preloadingsorbet-runtime,extend/module, orrubocop/test_prof.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
Library/Homebrew/utils/rubocop.rb |
Removes manual rubocop/test_prof require so RuboCop plugin loading is solely config-driven. |
Library/Homebrew/test/rubocop_spec.rb |
Simplifies the spawned RuboCop invocation by removing preloads that were previously used as workarounds. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.