Skip to content

Conversation

@mensfeld
Copy link
Collaborator

@mensfeld mensfeld commented Jan 25, 2026

Summary

  • Move yard-lint to a separate Gemfile.lint to isolate non-execution/linting dependencies
  • Add Gemfile.lint pattern to Renovate's bundler managerFilePatterns for automatic dependency updates

Test plan

  • Confirm Renovate detects and tracks Gemfile.lint

Summary by CodeRabbit

  • Chores
    • Reorganized linting dependencies by introducing a dedicated lint configuration file and moving the linting tool to this separate configuration.
    • Updated dependency management to recognize the new lint configuration file alongside existing patterns.

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

Move yard-lint to Gemfile.lint to isolate non-execution dependencies
from the main Gemfile. This keeps linting dependencies independent
while still being tracked by Renovate.
@coderabbitai
Copy link

coderabbitai bot commented Jan 25, 2026

📝 Walkthrough

Walkthrough

The pull request reorganizes yard-lint dependency management by removing it from Gemfile's test group, creating a dedicated Gemfile.lint configuration file, and updating renovate.json to recognize the new manifest pattern.

Changes

Cohort / File(s) Summary
Lint Dependency Separation
Gemfile, Gemfile.lint
Removed gem 'yard-lint' from Gemfile test group; created new Gemfile.lint with yard-lint dependency and frozen string literal pragma. Isolates linting dependencies from primary test dependencies.
Renovate Configuration
renovate.json
Added bundler managerFilePatterns entry for Gemfile.lint to enable dependency management tracking of the new manifest.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • phstc

Poem

🐰 With yard-lint seeking greener pastures,
A Gemfile.lint now holds its chapters,
Renovate nods—all's understood,
Dependencies organized, as they should! 📋

🚥 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 accurately summarizes the main change: extracting the yard-lint linting tool from the main Gemfile into a separate Gemfile.lint file, which aligns perfectly with the PR's primary objective.
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

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

@mensfeld mensfeld merged commit a69ce73 into main Jan 25, 2026
21 checks passed
@mensfeld mensfeld deleted the extract-linting-gemfile branch January 25, 2026 11:58
@mensfeld mensfeld mentioned this pull request Jan 25, 2026
2 tasks
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.

2 participants