f44bf0fe22
ci: move packages/private under fw-general
...
This doesn't need dev-infra approval as its mostly testing utils.
2026-09-01 14:42:04 -07:00
f3ed75eb6b
ci: remove shared primitives pullapprove group
...
This group is no longer necessary as responsibilities for that are now spread amongst teams more equally.
2026-08-31 15:03:05 -07:00
555369960a
ci: add Alan as a zone.js reviewer
...
This updates the zone.js owners and adds alan-agius4 to compiler approvers.
2026-08-24 10:02:14 -07:00
d364c83c44
ci: exclude markdown files from requiring dev-infra review
...
613c51e
2026-08-18 09:38:45 -07:00
613c51e324
ci: add local skills to the fw-general scope
...
They were caught by dev-infra which isn't the best scope for the imho
2026-08-18 08:54:16 -07:00
28f323a578
ci: update github nick
...
ethan changed his nick, this was causing some pullapproves errors when a pr hit the primitives-shared scope.
2026-08-17 10:40:39 -07:00
51724b48d2
ci: add myself to size-tracking
...
I feel confortable reviewing those
2026-06-08 11:52:20 -07:00
a816848373
ci: update devtools reviewers list
...
Update the Angular DevTools reviewers list in .pullapprove.yml. Add eduhmc and csmick, remove hybrist, and change milomg to approval-only.
2026-06-08 10:59:00 -07:00
95034a7b92
ci: mark devversion as unavailable
...
Currently OOO and I don't want reviews to be necessarily stuck for too long.
2026-05-12 10:47:09 -07:00
43a8df9520
ci: update pullapprove
...
This removes thePunderWoman from active review requests, but leaves passive on.
2026-05-11 12:37:45 -07:00
2b624580e5
ci: update PullApprove config
...
This commit updates the PullApprove config to keep myself as a reviewer, but do not auto-assign PR reviews.
2026-04-30 16:13:18 -07:00
7f3f3d7da1
ci: remove remainings of saucelabs tests
...
Those haven't been used for a while.
2026-04-22 14:41:03 -07:00
bf8105ef76
ci: exclude test files from primitives review
...
Those files aren't synced into G3 and shouldn't require external reviews
2026-04-01 20:27:26 +02:00
543753f551
ci: correct capitalization of MarkTechson in pullapprove
...
Correct the capitalization of MarkTechson's username to match their GitHub profile, ensuring proper evaluation by PullApprove.
2026-03-23 14:50:19 -07:00
bf93ff83d8
ci: fix exclusion of manual_api_docs
...
On top of #67253 , dev-infra should be required for docs files
2026-03-09 16:27:43 -07:00
0cd00b91ef
ci: update pullapprove for manual_api docs
...
The markdown files shouldn't require dev-infra approval.
2026-02-24 15:23:30 -08:00
0af9e3e9ee
ci: remove mmalerba from reviewers
...
remove mmalerba from reviewers
2026-02-23 10:18:56 -08:00
d7ddebca90
ci: update ng-dev config and pullapprove
...
This updates the pullapprove to remove the auto labeling of requires: TGP and instead moves it to the ng-dev config for pr updates.
2026-02-20 11:02:10 -08:00
29a17ed791
docs: Adds new agent skills to Angular
...
The first agent skill is a workflow for building a new Angular app from scratch. Using this skill agents will use Angular tooling for code generation where appropriate. Also includes important hints.
2026-02-13 10:07:54 -08:00
4937d162f5
ci: update pullapprove reviewer list
...
This cleans up the pullapprove reviewers.
2026-02-11 14:00:01 -08:00
50e74add52
build: add skills in antigravity
...
Adds a symlink `.agent/skills` to `.gemini/skills`. This will allow
antrigravity which searches under `.agent` to see them.
2026-02-11 13:39:52 -08:00
d019ee3c0b
ci: add myself to pull approve dev-infra reviewers
...
Add myself to the dev-infra reviewers.
2026-02-09 12:22:18 -08:00
c69b67147a
ci: add Charles & Doug as optional adev reviewers
...
Since we also have tooling docs...
2026-01-02 08:14:37 +01:00
25bf417d57
ci: include dev-app files from pullapprove checks
...
At dev-app to pullaprove
2025-12-01 19:24:22 +01:00
98ceb0a80d
ci: update pullapprove reviewers
...
Removing Minko from the pool of automatically picked reviewers
2025-11-24 13:25:26 -05:00
9dd7cc13ca
ci: adding myself as optional to the compiler scope
...
For whenever I feel confortable approving compiler PRs like diagnostics
2025-11-24 12:18:26 -05:00
fe2cc7967e
ci: use hardcoded global approvers group
...
Use a single hardcoded, within the pullapprove config file, group for global approvers instead of leveraging a github team as a group
2025-11-13 12:16:48 -08:00
50ee2072f6
ci: update handle for jkrems/hybrist
...
The username was changed earlier today and the old username is no
longer in use.
2025-11-03 19:43:52 +00:00
497dce962d
ci: update pullapprove config ( #64086 )
...
Update the pullapprove config to include vscode-ng-language-service
PR Close #64086
2025-09-25 14:34:41 -04:00
950ffd001e
build: remove yarn from integration tests, switch to pnpm ( #63902 )
...
This commit removes Yarn as a package manager for integration tests and migrates to pnpm. This change aims to standardize package management across the project, leveraging pnpm's efficiency and consistent behavior for dependency resolution and installation.
PR Close #63902
2025-09-18 14:39:51 +00:00
4b2cc12bc4
ci: remove myself from unavailability ( #63863 )
...
Removes myself from unavailability.
PR Close #63863
2025-09-17 14:27:42 +00:00
de8be9784d
ci: update pullapprove public-api rules ( #63737 )
...
This reduces the required reviews to 1 and requested reviewers to 2 to reduce friction.
PR Close #63737
2025-09-15 17:03:42 +00:00
16fa983989
ci: add more team members as approvers ( #63495 )
...
adds amishne, leonsenft, mattrbeck to approve but not receive review
requests
PR Close #63495
2025-08-29 15:12:13 -07:00
983cc2f6fd
ci: add myself to review fw-general ( #63337 )
...
Also adding myself to be assigned some adev PRs
PR Close #63337
2025-08-22 12:27:36 -07:00
6c8d000016
ci: mark myself as unavailable during OOO ( #63214 )
...
Marks myself as unavailable during OOO to not block reviews :)
PR Close #63214
2025-08-18 12:53:19 +02:00
82cc576e6f
build: use pnpm as the package manager instead of yarn ( #62924 )
...
Use pnpm instead of yarn as the package manager and interaction tool for the repo
PR Close #62924
2025-07-31 22:06:27 +00:00
9888fc4966
build: remove the esm interop patches and tooling ( #62879 )
...
Remove esm interop patches and tooling from previous toolchain
PR Close #62879
2025-07-29 17:25:35 +00:00
743b928331
ci: update renovate reviewers ( #62761 )
...
This restores the framework-team availability, since the setting was not working as intended.
PR Close #62761
2025-07-23 07:24:13 -04:00
b0effa1cc3
ci: use new size tracking in integration tests ( #62650 )
...
Use the new size tracking within integration tests
PR Close #62650
2025-07-17 13:56:22 -04:00
bece163f87
ci: add to public-api approvers list ( #62591 )
...
This appends reviewers to the approvers list.
PR Close #62591
2025-07-11 10:43:27 -07:00
624195706d
ci: adjust renovate teams ( #62578 )
...
This switches framework team to approve but not requested, and adds the currently active caretaker group to the teams for renovate.
PR Close #62578
2025-07-10 12:52:31 -07:00
c682dfffcd
ci: add gemini code assist config ( #62560 )
...
Configures gemini to not interact with new pull requests, only review on
demand
PR Close #62560
2025-07-09 12:42:45 -07:00
aad1e747d6
ci: update pullapprove reviewers ( #62448 )
...
This sets some people to be reviewers, but not actively requested to account for availability.
PR Close #62448
2025-07-03 09:15:19 +00:00
53575390f4
ci: add jkrems to pullapprove (public-api) ( #62340 )
...
Allow jkrems to (co-)approve public-api changes.
PR Close #62340
2025-06-27 18:37:00 +00:00
1bf1fb2ad7
ci: Update pullapprove ownership for compiler and zonejs ( #62261 )
...
This updates the zone.js owners and adds thePunderWoman to compiler approvers.
PR Close #62261
2025-06-25 12:46:40 +00:00
579618d6d6
build: remove packages/bazel ( #61843 )
...
Removes the `@angular/bazel` package as we no longer need it. Yay!
It's replaced by `rules_angular`.
PR Close #61843
2025-06-04 09:13:41 +00:00
2f0ecfc8ef
refactor: expand DevTools reviewer list ( #61709 )
...
Updates the current list of DevTools reviewers to make a few small changes.
1. Adds `hawkgs` and `milomg`.
2. Removes the leading `~` from `JeanMeche` to include him in automatic review requests.
3. Adds `jkrems` for additional Googler coverage in automatic review requests.
4. Removes `mgechev` and `MarkTechson` who haven't been involved in daily DevTools development for a while.
5. Adds the leading `~` to `devversion` and `josephperrott` to remove them from automatic review requests while leaving them with access for "break glass" situations.
Note that the leading `~` means that the individual may approve PRs and satisfy the group's requirement, but PullApprove will not automatically request a review from them. See https://v3-docs.pullapprove.com/config/reviewers/#reviewers-that-dont-want-review-requests .
PR Close #61709
2025-05-30 09:56:09 -04:00
032b802f54
build: remove irrelevant madge circular deps tests ( #61156 )
...
We don't need this tooling anymore because we are already validating
that there are no circular dependencies via the `ng-dev` tooling that
checks `.ts` files directly.
Also these tests never actually failed to my knowledge.
PR Close #61156
2025-05-07 11:28:59 -07:00
af67334fe5
ci: add hawkgs to pullapprove ( #60536 )
...
Add hawkgs to pullaprove for angular-dev.
PR Close #60536
2025-03-26 20:39:31 -07:00
3f0f2e65a8
ci: update shared primitives reviewers list ( #60447 )
...
This tightens up the list of reviewers for any shared primitives code changes.
PR Close #60447
2025-03-19 08:37:40 +01:00
Matthieu Riegler