####################################################################################

####################################################################################

# Angular Code Ownership #

####################################################################################

####################################################################################

#

# Configuration of code ownership and review approvals for the angular/angular repo.

#

# More info: https://docs.pullapprove.com/

#

# =========================================================

# General rules / philosophy

# =========================================================

#

# - We trust that people do the right thing and won't approve changes they don't feel confident reviewing

# - We enforce that only approved PRs are merged ensuring that unreviewed code isn't accidentally merged

# - We distribute approval rights as much as possible to help us scale better

# - A small number of file groups have very limited number of reviewers because incorrect changes to the files they guard would have serious consequences (e.g. security, public api)

#

# Configuration nuances:

#

# - This configuration works in conjunction with the protected branch settings that require all changes to be made via pull requests with at least one approval.

# - This approval can come from an appropriate codeowner, or any repo collaborator (person with write access) if the PR is authored by a codeowner.

# - All groups whose pullapprove rules are matched will be required for overall approval.

#

# NOTE:

# In the case of emergency, the repo administrators which include the current angular caretaker

# can bypass this reviewer approval requirement, this is expected as a last resort and to be

# done exceedingly sparingly.

####################################################################################

# GitHub usernames

####################################################################################

# See reviewer list under `required-minimum-review` group. Team member names and

# usernames are managed there.

version: 3

#availability:

# users_unavailable: []

# Meta field that goes unused by PullApprove to allow for defining aliases to be

# used throughout the config.

meta:

# Note: Because all inactive groups start as pending, we are only checking pending and rejected active groups.

no-groups-above-this-pending: &no-groups-above-this-pending len(groups.active.pending.exclude("required-minimum-review")) == 0

no-groups-above-this-rejected: &no-groups-above-this-rejected len(groups.active.rejected.exclude("required-minimum-review")) == 0

defaults: &defaults

reviews:

# Authors provide their approval implicitly, this approval allows for a reviewer

# from a group not to need a review specifically for an area of the repository

# they own. This is coupled with the `required-minimum-review` group which requires

# that all PRs are reviewed by at least one team member who is not the author of

# the PR.

author_value: 1

# turn on 'draft' support

# https://docs.pullapprove.com/config/github-api-version/

# https://developer.github.com/v3/previews/#draft-pull-requests

github_api_version: 'shadow-cat-preview'

# https://docs.pullapprove.com/config/overrides/

# Note that overrides are processed in order.

overrides:

# For PRs which are still being worked on, either still in draft mode or indicated through WIP in

# title or label, PullApprove stays in a pending state until its ready for review.

- if: "draft or 'WIP' in title or 'PR state: WIP' in labels"

status: pending

explanation: 'Waiting to send reviews as PR is WIP'

# Disable PullApprove on specific PRs by adding the `PullApprove: disable` label

- if: "'PullApprove: disable' in labels"

status: success

explanation: "PullApprove skipped because of 'PullApprove: disable' label"

# If no file matching based groups are active, report this pull request as failing. Most likely,

# the PR author would need to update the PullApprove config, or create new group.

- if: len(groups.active.exclude("required-minimum-review").exclude("global-*")) == 0 and len(groups.approved.include("global-*")) == 0

status: failure

explanation: 'At least one group must match this PR. Please update an existing review group, or create a new group.'

# If any global approval is given the status should be passing.

- if: len(groups.approved.include("global-approvers")) == 1

status: success

explanation: 'Passing as globally approved by global approvers'

groups:

# =========================================================

# Framework: Compiler

# =========================================================

fw-compiler:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'packages/compiler/**/{*,.*}',

'packages/examples/compiler/**/{*,.*}',

'packages/compiler-cli/**/{*,.*}',

'packages/language-service/**/{*,.*}',

])

reviewers:

users:

- ~alxhub

- AndrewKushnir

- atscott

- crisbeto

- devversion

- thePunderWoman

- kirjs

- JoostK

- ~amishne

- ~leonsenft

- ~mattrbeck

- ~JeanMeche

# =========================================================

# Framework: General (most code in our packages)

# =========================================================

fw-general:

<<: *defaults

conditions:

- >

contains_any_globs(files.exclude('packages/core/primitives/*'), [

'contributing-docs/public-api-surface.md',

'dev-app/**/{*,.*}',

'integration/**/{*,.*}',

'modules/**/{*,.*}',

'packages/animations/**/{*,.*}',

'packages/benchpress/**/{*,.*}',

'packages/common/**/{*,.*}',

'packages/core/**/{*,.*}',

'packages/docs/**/{*,.*}',

'packages/elements/**/{*,.*}',

'packages/examples/**/{*,.*}',

'packages/forms/**/{*,.*}',

'packages/localize/**/{*,.*}',

'packages/misc/**/{*,.*}',

'packages/platform-browser/**/{*,.*}',

'packages/platform-browser-dynamic/**/{*,.*}',

'packages/platform-server/**/{*,.*}',

'packages/ssr/**/{*,.*}',

'packages/router/**/{*,.*}',

'packages/service-worker/**/{*,.*}',

'packages/upgrade/**/{*,.*}',

'vscode-ng-language-service/**/{*,.*}',

])

reviewers:

users:

- ~alxhub

- AndrewKushnir

- atscott

- crisbeto

- devversion

- kirjs

- thePunderWoman

- ~pkozlowski-opensource

- JeanMeche

- ~amishne

- ~leonsenft

- ~mattrbeck

# =========================================================

# Framework: Security-sensitive files which require extra review

# =========================================================

fw-security:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'packages/core/src/sanitization/**/{*,.*}',

'packages/core/test/linker/security_integration_spec.ts',

'packages/compiler/src/schema/**/{*,.*}',

'packages/platform-browser/src/security/**/{*,.*}',

'packages/tsconfig-tsec-base.json',

'packages/tsec-exemption.json',

'tools/tsec.bzl',

'adev/src/content/guide/security.md',

'adev/src/content/examples/security/**/{*,.*}',

])

reviewers:

users:

- ~alxhub

- josephperrott

- ~pkozlowski-opensource

- alan-agius4

reviews:

request: -1 # request reviews from everyone

required: 2 # require at least 2 approvals

reviewed_for: required

# =========================================================

# zone.js

# =========================================================

zone-js:

<<: *defaults

conditions:

- >

contains_any_globs(files.exclude('yarn.lock'), [

'packages/zone.js/**/{*,.*}',

])

reviewers:

users:

- ~JiaLiPassion

- ~alxhub

- AndrewKushnir

- atscott

- crisbeto

- devversion

- kirjs

- thePunderWoman

- ~pkozlowski-opensource

- ~amishne

- ~leonsenft

- ~mattrbeck

# =========================================================

# Tooling: Compiler API shared with Angular CLI

#

# Changing this API might break Angular CLI, so we require

# the CLI team to approve changes here.

# =========================================================

tooling-cli-shared-api:

conditions:

- >

contains_any_globs(files, [

'packages/compiler-cli/private/tooling.ts',

'packages/localize/tools/index.ts'

])

reviewers:

users:

- alan-agius4

- clydin

- dgp1130

reviews:

request: -1 # request reviews from everyone

required: 2 # require at least 2 approvals

reviewed_for: required

# =========================================================

# Documentation content

# =========================================================

angular-dev:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'adev/**/{*,.*}',

'tools/manual_api_docs/blocks/*.md',

'tools/manual_api_docs/elements/*.md',

])

reviewers:

users:

- alan-agius4

- ~alxhub

- AndrewKushnir

- atscott

- bencodezen

- crisbeto

- kirjs

- JeanMeche

- thePunderWoman

- devversion

- josephperrott

- ~pkozlowski-opensource

- ~mgechev

- MarkTechson

- ~hawkgs

- ~amishne

- ~leonsenft

- ~mattrbeck

- ~clydin

- ~dgp1130

# =========================================================

# Angular DevTools

# =========================================================

devtools:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'devtools/**/{*,.*}',

])

reviewers:

users:

- AleksanderBodurri

- ~devversion

- dgp1130

- hawkgs

- hybrist

- ~josephperrott

- JeanMeche

- milomg

# =========================================================

# Dev-infra

# =========================================================

dev-infra:

<<: *defaults

conditions:

- >

contains_any_globs(files

.exclude('.pullapprove.yml')

.exclude('tools/manual_api_docs/blocks/*.md')

.exclude('tools/manual_api_docs/elements/*.md'),

[

'{*,.*}',

'.agent/**/{*,.*}',

'.devcontainer/**/{*,.*}',

'.github/**/{*,.*}',

'.husky/**/{*,.*}',

'.gemini/**/{*,.*}',

'.ng-dev/**/{*,.*}',

'.vscode/**/{*,.*}',

'contributing-docs/*.md',

'contributing-docs/images/**/{*,.*}',

'goldens/{*,.*}',

'goldens/public-api/manage.js',

'modules/{*,.*}',

'packages/{*,.*}',

'packages/examples/test-utils/**/{*,.*}',

'packages/private/**/{*,.*}',

'packages/examples/{*,.*}',

'scripts/**/{*,.*}',

'third_party/**/{*,.*}',

'tools/contributing-stats/**/{*,.*}',

'tools/gulp-tasks/**/{*,.*}',

'tools/legacy-saucelabs/**/{*,.*}',

'tools/manual_api_docs/**/{*,.*}',

'tools/pnpm-patches/**/{*,.*}',

'tools/rxjs/**/{*,.*}',

'tools/saucelabs-daemon/**/{*,.*}',

'tools/saucelabs/**/{*,.*}',

'tools/symbol-extractor/**/{*,.*}',

'tools/testing/**/{*,.*}',

'tools/tslint/**/{*,.*}',

'tools/utils/**/{*,.*}',

'tools/yarn/**/{*,.*}',

'tools/{*,.*}',

'**/*.bzl'

])

- author not in ["angular-robot"]

reviewers:

users:

- alan-agius4

- devversion

- josephperrott

# =========================================================

# Renovate Changes

# =========================================================

renovate-changes:

<<: *defaults

conditions:

- author in ["angular-robot"]

reviewers:

users:

- ~alan-agius4

teams:

- angular-caretaker

- framework-team

# =========================================================

# Public API

# =========================================================

public-api:

<<: *defaults

conditions:

- *no-groups-above-this-pending

- *no-groups-above-this-rejected

- >

contains_any_globs(files.exclude("goldens/public-api/manage.js"), [

'goldens/public-api/**/{*,.*}',

])

reviewers:

users:

- AndrewKushnir

- ~alxhub

- atscott

- thePunderWoman

- ~pkozlowski-opensource

- kirjs

- crisbeto

- devversion

- JeanMeche

- ~hybrist

- ~iteriani

- ~tbondwilkinson

- ~rahatarmanahmed

- ~amishne

- ~leonsenft

- ~mattrbeck

reviews:

request: 2

required: 1

reviewed_for: required

# ================================================

# Size tracking

# ================================================

size-tracking:

<<: *defaults

conditions:

- *no-groups-above-this-pending

- *no-groups-above-this-rejected

- >

contains_any_globs(files, [

'integration/**/size.json'

])

reviewers:

users:

- ~alxhub

- AndrewKushnir

- atscott

- kirjs

- thePunderWoman

- ~pkozlowski-opensource

- ~amishne

- ~leonsenft

- ~mattrbeck

reviews:

request: 2 # Request reviews from 2 people

required: 1 # Require that 1 person approve

reviewed_for: required

# =========================================================

# Agent Skills

# =========================================================

agent-skills:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'skills/{*,.*}',

'skills/**/{*,.*}',

])

reviewers:

users:

- marktechson

- kirjs

- ~JeanMeche

- ~dgp1130

####################################################################################

# Special Cases

####################################################################################

# =========================================================

# Code Ownership

# =========================================================

code-ownership:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'.pullapprove.yml'

])

reviewers:

users:

- ~alxhub

- AndrewKushnir

- andrewseguin

- dgp1130

- thePunderWoman

- josephperrott

# =========================================================

# Primitives

# =========================================================

# Angular team required reviews

primitives:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'packages/core/primitives/**/{*,.*}',

])

reviewers:

users:

- ~pkozlowski-opensource # Pawel Kozlowski

- ~alxhub # Alex Rickabaugh

- thePunderWoman # Jessica Janiuk

- AndrewKushnir # Andrew Kushnir

- atscott # Andrew Scott

# External team required reviews

primitives-shared:

<<: *defaults

conditions:

- >

contains_any_globs(files, [

'packages/core/primitives/**/{*,.*}',

])

reviewers:

users:

- csmick # Cameron Smick

- mturco # Matt Turco

- iteriani # Thomas Nguyen

- tbondwilkinson # Tom Wilkinson

- rahatarmanahmed # Rahat Ahmed

- ENAML # Ethan Cline

####################################################################################

# Override managed result groups

#

# Groups which are only used to determine the value of an override are managed at

# the bottom of the list as they will set a status on the PR directly, they

# therefore can always be process last without concern.

####################################################################################

# =========================================================

# Global Approvers

#

# All reviews performed for global approvals require using

# the `Reviewed-for:` specifier to set the approval

# specificity as documented at:

# https://docs.pullapprove.com/reviewed-for/

# =========================================================

global-approvers:

type: optional

reviewers:

users:

- alxhub

- josephperrott

reviews:

request: 0

required: 1

reviewed_for: required

# =========================================================

# Require review on all PRs

#

# All PRs require at least one review. This rule will not

# request any reviewers, however will require that at least

# one review is provided before the group is satisfied.

# =========================================================

required-minimum-review:

reviews:

request: 0 # Do not request any reviews from the reviewer group

required: 1 # Require that all PRs have approval from at least one of the users in the group

author_value: 0 # The author of the PR cannot provide an approval for themself

reviewed_for: ignored # All reviews apply to this group whether noted via Reviewed-for or not

reviewers:

teams:

# Any member of the team can provide a review to perform the minimum review required

- team