Skip to content

Extend Apps Infra CODEOWNERS routing to CI and toolchain - #694

Merged
mokagio merged 4 commits into
trunkfrom
mokagio/ainfra-2437-codeowners-widen
Jul 30, 2026
Merged

Extend Apps Infra CODEOWNERS routing to CI and toolchain#694
mokagio merged 4 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 #689 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

Extends GitHub CODEOWNERS routing in this Android SDK repo so changes to broader Apps Infra-related surfaces (Fastlane, CI/automation, and toolchain pin files) request review from @Automattic/apps-infra-tooling, aligning with the intent from #689.

Changes:

  • Add ownership for fastlane/ to route release automation updates.
  • Add ownership for CI/automation surfaces: .buildkite/, .github/workflows/, .github/dependabot.yml, and Dangerfile*.
  • Add ownership for toolchain pin file .java-version.

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

@wpmobilebot

wpmobilebot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

🚧 Demo app build will be available soon

@mokagio mokagio added the infra Changes related to the infrastructure label Jun 12, 2026
@mokagio
mokagio requested a review from a team June 12, 2026 00:22
Route a8c configure secret material (.configure-files/) to
apps-infra-tooling, per AINFRA-2437.
@mokagio
mokagio enabled auto-merge June 15, 2026 08:34
@mokagio
mokagio disabled auto-merge June 15, 2026 08:34
@mokagio
mokagio enabled auto-merge June 15, 2026 08:34
@mokagio
mokagio merged commit d07e72c into trunk Jul 30, 2026
12 of 14 checks passed
@mokagio
mokagio deleted the mokagio/ainfra-2437-codeowners-widen branch July 30, 2026 14:28
Comment thread .github/CODEOWNERS
.java-version @Automattic/apps-infra-tooling

# Secrets
.configure-files/ @Automattic/apps-infra-tooling

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.

@mokagio I didn't want to block the merging for this, but checking now just in case: is .configure-files/ added here still going to be used or is it just for a transitional period?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Changes related to the infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants