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 rubocop without preloading sorbet-runtime, extend/module, or rubocop/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.