Skip to content

chore: align repo baseline (CODEOWNERS v4, renovate onboarding, baseline workflows) - #3

Merged
mathieudi merged 1 commit into
mainfrom
chore/alignment-baseline
Jul 30, 2026
Merged

chore: align repo baseline (CODEOWNERS v4, renovate onboarding, baseline workflows)#3
mathieudi merged 1 commit into
mainfrom
chore/alignment-baseline

Conversation

@mathieudi

Copy link
Copy Markdown
Contributor

Creates CODEOWNERS from the v4 skeleton with platform as owner and devops
re-owning the meta files.

Onboards Renovate with the default preset and an explicit
:reviewer(team:platform), and adds the claude-code, code-review and
request-devops-review consumers.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com


Part of the org-wide repo-alignment rollout

Baseline definition and rationale live in the alignment plan; the same shape has already been
validated on test-service-ci, test-nextjs-ci, template-service and template-library.

  • Required check contexts are unchanged unless this PR says otherwise — verified mechanically
    against this repo's live branch protection before opening.
  • merge_group triggers are inert until the merge queue is actually enabled for this repo
    (a later, separate step). Adding them first is what makes that switch safe.
  • Typed chore/chore(ci) on purpose: @side/release-config maps feat → minor, and repo
    configuration changes must not cut releases or trigger deploys.
  • request-devops-review covers .github/**, .yarnrc.yml and .yarn/**. Bot authors are
    skipped, so Renovate keeps auto-merging; human edits to CI or Yarn config request a devops review.
    (.yarn/ holds the Yarn binary, plus plugins/patches where present — .yarnrc.yml only points
    at them.)

@mathieudi
mathieudi requested review from a team, alexsupp and hershmire and removed request for a team July 29, 2026 17:11
…ine workflows)

Creates CODEOWNERS from the v4 skeleton with platform as owner and devops
re-owning the meta files.

Onboards Renovate with the default preset and an explicit
:reviewer(team:platform), and adds the claude-code, code-review and
request-devops-review consumers.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
CODEOWNERS keeps only the bare `package.json` / `yarn.lock` carve-outs. A bare pattern
already matches at any depth, so `**/package.json` / `**/yarn.lock` were redundant.
Verified empirically on reside-eng/test-monorepo rather than assumed: a PR touching only
`packages/hello/package.json` had NO code owner auto-requested, while a PR touching only
`packages/hello/src/index.ts` in the same directory requested @reside-eng/platform-tools.
@mathieudi
mathieudi force-pushed the chore/alignment-baseline branch from 99afdec to 5d99b17 Compare July 30, 2026 11:35
@side-review-request
side-review-request Bot requested review from a team and prescottprue and removed request for a team July 30, 2026 11:35
@mathieudi
mathieudi merged commit 28bd7b8 into main Jul 30, 2026
1 check passed
@mathieudi
mathieudi deleted the chore/alignment-baseline branch July 30, 2026 14:49
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