Skip to content

Comments

Reduce team review request noise from CODEOWNERS#204

Merged
edmorley merged 1 commit intomainfrom
edmorley/update-codeowners
Feb 1, 2024
Merged

Reduce team review request noise from CODEOWNERS#204
edmorley merged 1 commit intomainfrom
edmorley/update-codeowners

Conversation

@edmorley
Copy link
Member

@edmorley edmorley commented Feb 1, 2024

The CODEOWNERS file has been adjusted to request review from the primary repository maintainer for high-traffic files that are typically updated via automation, rather than requesting review from the whole team.

For more information, see:
heroku/webapp-runner#423

GUS-W-14941625.

The `CODEOWNERS` file has been adjusted to request review from the
primary repository maintainer for high-traffic files that are
typically updated via automation, rather than requesting review from
the whole team.

See also:
heroku/webapp-runner#423

GUS-W-14941625.
@edmorley edmorley self-assigned this Feb 1, 2024
@edmorley edmorley marked this pull request as ready for review February 1, 2024 16:09
@edmorley edmorley requested a review from a team as a code owner February 1, 2024 16:09
@edmorley edmorley enabled auto-merge (squash) February 1, 2024 16:10
@edmorley edmorley requested review from Malax and removed request for a team February 1, 2024 16:10
@edmorley edmorley merged commit a8b0edd into main Feb 1, 2024
@edmorley edmorley deleted the edmorley/update-codeowners branch February 1, 2024 16:10
@edmorley
Copy link
Member Author

edmorley commented Feb 1, 2024

@Malax Ah auto-merge insta-merged even with the pending review, since "require approval" is off (which is expected for most of the getting started guide repos). I'm happy to make changes to this if needed, if you leave comments on the PR.

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