Move rubocop to Gemfile.lint

[mensfeld]

Summary

• Move rubocop from main Gemfile to Gemfile.lint to consolidate all linting tools in one place
• Update Gemfile.lint.lock with rubocop dependencies

This is a follow-up to #951 which extracted yard-lint but missed rubocop.

Test plan

• Verify BUNDLE_GEMFILE=Gemfile.lint bundle exec rubocop works
• Verify main bundle install still works without rubocop

Summary by CodeRabbit

• Chores
  • Reorganized code linting configuration across development dependency files.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request relocates the rubocop gem dependency from the main Gemfile's development group to a separate Gemfile.lint file, consolidating linting-related dependencies into a dedicated configuration file.

Changes

Cohort / File(s)	Summary
Linting Dependencies Reorganization Gemfile, Gemfile.lint	Removed rubocop gem from :development group in Gemfile; added rubocop declaration to new/separate Gemfile.lint with linting section comment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Extract linting tools to separate Gemfile #951: Follows the same pattern of moving linting gems from main Gemfile to a separate Gemfile.lint file (yard-lint dependency).
• Cleanup redundant gems in the Gemfiles #803: Directly modifies rubocop declarations in Gemfile configuration.

Suggested reviewers

• phstc

Poem

🐰 Hop, hop! A linting gem takes flight,
From Gemfile to lint—what a sight!
Organized, clean, no clutter in view,
Separate concerns make code smell new! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Move rubocop to Gemfile.lint' clearly and concisely summarizes the main change: relocating the rubocop gem from the main Gemfile to Gemfile.lint for better dependency organization.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}