Skip to content

Group dependabot bundler updates using multi-ecosystem-groups#994

Merged
tagliala merged 3 commits into
masterfrom
copilot/group-gemfiles-for-rails
Mar 6, 2026
Merged

Group dependabot bundler updates using multi-ecosystem-groups#994
tagliala merged 3 commits into
masterfrom
copilot/group-gemfiles-for-rails

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 5, 2026

Reduce dependabot PR noise by grouping all bundler updates (root + per-Rails-version gemfiles) into a single PR using multi-ecosystem-groups.

  • Added top-level multi-ecosystem-groups.bundler with monthly schedule
  • Replaced per-entry schedule and groups blocks with multi-ecosystem-group: bundler and required patterns field
  • All existing ignore rules and versioning-strategy: lockfile-only preserved
  • GitHub Actions config unchanged

Inspired by activeadmin/activeadmin#8952


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Reduce dependabot PRs by grouping gemfiles Group dependabot bundler updates using multi-ecosystem-groups Mar 5, 2026
@tagliala
Copy link
Copy Markdown
Contributor

tagliala commented Mar 6, 2026

@javierjulio actually my comment related to potential issues belongs here :(

s.required_ruby_version = '>= 3.1'

This is still supporting 3.1

I'll drop it, cleanup things, and submit a new PR for review

Use the multi-ecosystem-groups feature to group all bundler updates
(root Gemfile + all Rails version gemfiles) into a single PR instead
of separate PRs per directory. Schedule changed to weekly.

Co-authored-by: tagliala <556268+tagliala@users.noreply.github.com>
@tagliala tagliala force-pushed the copilot/group-gemfiles-for-rails branch from 0debfe5 to 310c190 Compare March 6, 2026 20:09
@tagliala tagliala marked this pull request as ready for review March 6, 2026 20:10
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.93%. Comparing base (81e0472) to head (e5c3811).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #994   +/-   ##
=======================================
  Coverage   98.93%   98.93%           
=======================================
  Files          14       14           
  Lines         563      563           
=======================================
  Hits          557      557           
  Misses          6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces Dependabot PR noise by consolidating all bundler update entries (root gemfile + per-Rails-version gemfiles) into a single grouped PR using the multi-ecosystem-groups feature. Instead of each bundler directory having its own schedule and groups block, they all reference a single top-level multi-ecosystem-group: bundler definition.

Changes:

  • Added a top-level multi-ecosystem-groups.bundler block with a weekly schedule.
  • Removed the per-entry schedule and groups blocks from all five bundler entries, replacing them with multi-ecosystem-group: bundler.
  • All existing ignore rules and versioning-strategy: lockfile-only settings are preserved.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/dependabot.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@tagliala tagliala left a comment

Choose a reason for hiding this comment

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

@copilot I'm getting this:

The property '#/updates/1/patterns' is required when 'multi-ecosystem-group' is set.

…roup

Co-authored-by: tagliala <556268+tagliala@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 6, 2026

@copilot I'm getting this:

The property '#/updates/1/patterns' is required when 'multi-ecosystem-group' is set....

Added the required patterns field to all bundler entries that use multi-ecosystem-group. Fixed in e5c3811.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tagliala tagliala merged commit 204a349 into master Mar 6, 2026
28 checks passed
@tagliala tagliala deleted the copilot/group-gemfiles-for-rails branch March 6, 2026 20:21
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.

3 participants