Skip to content

Extend Apps Infra CODEOWNERS routing to CI and toolchain#1393

Open
mokagio wants to merge 2 commits into
trunkfrom
mokagio/ainfra-2437-codeowners-widen
Open

Extend Apps Infra CODEOWNERS routing to CI and toolchain#1393
mokagio wants to merge 2 commits into
trunkfrom
mokagio/ainfra-2437-codeowners-widen

Conversation

@mokagio

@mokagio mokagio commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Extends the CODEOWNERS routing from #1370 beyond the Ruby dependency surface to the broader Apps Infra surface — CI config, automation, and toolchain pins — still routed to @Automattic/apps-infra-tooling. Only paths that exist in this repo are included. See AINFRA-2437.

The routing is path-based, so it applies to any PR touching these files, not just Dependabot's; deliberate, since the retired dependabot.yml reviewers key has no source-scoped replacement.


Posted by Claude Code (Fable 5) on behalf of @mokagio.

Widens the Ruby-only routing block to the broader Apps Infra
surface (CI config, automation, toolchain pins), per the
dependabot-bootstrap campaign (AINFRA-2437).
Only paths present in this repo are included.

---

Generated with the help of Claude Code, https://code.claude.com

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 11, 2026 05:35
@mokagio mokagio self-assigned this Jun 11, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 extends .github/CODEOWNERS coverage from the Ruby dependency surface to additional Apps Infra areas (Fastlane, CI/automation config, and toolchain/environment pin files) so changes to those paths are routed to @Automattic/apps-infra-tooling.

Changes:

  • Add CODEOWNERS routing for fastlane/.
  • Add CODEOWNERS routing for CI/automation paths under .buildkite/ and .github/workflows/, plus .github/dependabot.yml.
  • Add a CODEOWNERS entry intended for a Java toolchain pin file.

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

Comment thread .github/CODEOWNERS
Comment on lines +11 to +12
# Toolchain and environment pins
.java-version @Automattic/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.

Thanks — this one is working as intended, so I'm keeping the entry.

The .java-version pattern has no inner slash, so per gitignore/CODEOWNERS semantics it's unanchored and matches at any depth, not just the repo root. This repo does have a .java-version — at native/kotlin/.java-version — so the rule routes that file to @Automattic/apps-infra-tooling rather than being a no-op.

The "only existing paths are included" claim in the description holds under that any-depth reading: every line in the block corresponds to a file that exists somewhere in the tree (Gemfile*, .bundle/, .ruby-version, .java-version, .buildkite/, .github/workflows/, .github/dependabot.yml, fastlane/). The unanchored matching is deliberate so nested pins like this are covered without per-repo path juggling.

Posted by Claude Code (Fable 5) on behalf of @mokagio with approval.

@wpmobilebot

wpmobilebot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

XCFramework Build

This PR's XCFramework is available for testing. Add to your Package.swift:

.package(url: "https://github.com/automattic/wordpress-rs", branch: "pr-build/1393")

Built from 3fb49da

The repo's memo-validate-changelog check requires every PR to add a
CHANGELOG entry; the widen PR touched only CODEOWNERS and had none.

Co-Authored-By: Claude Code (Fable 5) <noreply@anthropic.com>
@mokagio mokagio requested a review from a team June 12, 2026 00:22
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