Skip to content

Route Dependabot Ruby reviews via CODEOWNERS - #22933

Merged
mokagio merged 3 commits into
trunkfrom
mokagio/ainfra-2437-dependabot-codeowners
Jun 2, 2026
Merged

Route Dependabot Ruby reviews via CODEOWNERS#22933
mokagio merged 3 commits into
trunkfrom
mokagio/ainfra-2437-dependabot-codeowners

Conversation

@mokagio

@mokagio mokagio commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Why

GitHub retired the dependabot.yml reviewers key on 2025-08-08 — Dependabot ignores it and defers to CODEOWNERS. The key was added before we caught the deprecation, so it's currently a no-op.

This adds CODEOWNERS entries routing future Dependabot Ruby PRs to apps-infra-tooling, and removes the dead key. Part of AINFRA-2437.

Note

CODEOWNERS is path-driven, so this also requests apps-infra-tooling on any human PR touching the Ruby tooling files (Gemfile*, .ruby-version, .bundle/, .rubocop*.yml) — intended.

How to test

GitHub validates CODEOWNERS on push; confirm no errors under the repo's CODEOWNERS check.


Posted by Claude (Opus 4.8) on behalf of @mokagio with approval.

Copilot AI review requested due to automatic review settings June 2, 2026 06:28
@mokagio mokagio self-assigned this Jun 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates Dependabot review routing for Ruby-related dependency/tooling changes by migrating from the deprecated dependabot.yml reviewers key to path-based CODEOWNERS rules.

Changes:

  • Removed the deprecated/no-op reviewers key from the Bundler section in .github/dependabot.yml.
  • Added CODEOWNERS entries to request reviews from @wordpress-mobile/apps-infra-tooling for Ruby tooling files (Gemfile*, .ruby-version, .bundle/, .rubocop*.yml).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/dependabot.yml Removes deprecated Dependabot reviewer configuration for Bundler updates.
.github/CODEOWNERS Adds ownership rules so Dependabot Ruby/tooling PRs request reviews from the tooling team.

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

Comment thread .github/CODEOWNERS
Comment on lines +5 to +8
Gemfile* @wordpress-mobile/apps-infra-tooling
.ruby-version @wordpress-mobile/apps-infra-tooling
.bundle/ @wordpress-mobile/apps-infra-tooling
.rubocop*.yml @wordpress-mobile/apps-infra-tooling

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I like how generic the pattern is. It will remain valid even if those files are moved to subfolders.


Posted by Claude (Opus 4.8) on behalf of @mokagio with approval.

@mokagio mokagio added this to the 26.9 milestone Jun 2, 2026
@wpmobilebot

wpmobilebot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress Android by scanning the QR code below to install the corresponding build.

App NameWordPress Android
Build TypeDebug
Versionpr22933-3f3ae94
Build Number1493
Application IDorg.wordpress.android.prealpha
Commit3f3ae94
Installation URL1iercj7t7o24g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

wpmobilebot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack Android by scanning the QR code below to install the corresponding build.

App NameJetpack Android
Build TypeDebug
Versionpr22933-3f3ae94
Build Number1493
Application IDcom.jetpack.android.prealpha
Commit3f3ae94
Installation URL7o0ql76c0oj1g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@mokagio
mokagio enabled auto-merge (squash) June 2, 2026 07:28
@mokagio
mokagio requested a review from a team June 2, 2026 07:28
@jkmassel
jkmassel force-pushed the mokagio/ainfra-2437-dependabot-codeowners branch from 7500794 to 3f3ae94 Compare June 2, 2026 21:36
@mokagio
mokagio merged commit edee3df into trunk Jun 2, 2026
17 of 19 checks passed
@mokagio
mokagio deleted the mokagio/ainfra-2437-dependabot-codeowners branch June 2, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants