Skip to content

chore(ci): fail PRs that modify labs-owned paths - #25285

Open
fcarreiro wants to merge 1 commit into
nextfrom
fc/labs-path-guard
Open

chore(ci): fail PRs that modify labs-owned paths#25285
fcarreiro wants to merge 1 commit into
nextfrom
fc/labs-path-guard

Conversation

@fcarreiro

Copy link
Copy Markdown
Contributor

Labs-owned code has moved to aztec-labs-eng/aztec-node; changes to it should land there, not here.

This adds a labs-ownership-guard job to ci3.yml that fails loudly ("This PR tries to modify labs-owned code") when a PR targeting next touches any labs-owned path. The path list mirrors what the root Makefile's fast-labs target builds:

  • yarn-project/
  • noir-projects/labs/
  • aztec-up/
  • docs/
  • playground/
  • release-image/
  • spartan/
  • labs-aztec-toolchain/

Notes:

  • Changed files come from the GitHub PR files API, so no checkout or diff-depth handling is needed.
  • Scoped to PRs targeting next — release/staging branch backports that still carry labs paths are unaffected.
  • labs-aztec-toolchain/ is included as labs-owned; easy to drop from the list if foundation still needs to touch it while the monorepo build depends on it.
  • No override label for now; one can be added if a legitimate cross-cutting change comes up.

Labs-owned code (the fast-labs side of the root Makefile: yarn-project,
noir-projects/labs, aztec-up, docs, playground, release-image, spartan,
labs-aztec-toolchain) has moved to aztec-labs-eng/aztec-node. Add a
labs-ownership-guard job to ci3.yml that fails any PR targeting next
that touches those paths.
@fcarreiro
fcarreiro marked this pull request as ready for review August 21, 2026 14:03
@fcarreiro
fcarreiro requested a review from charlielye as a code owner August 21, 2026 14:03
@fcarreiro
fcarreiro requested review from IlyasRidhuan and spalladino and removed request for charlielye August 21, 2026 14:05
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.

1 participant