Skip to content

Maintenance: add Powertools Core team to CODEOWNERS #2098

Closed
@dreamorosi

Description

@dreamorosi

Summary

In the past few weeks we have strengthened our branch protection so that all PRs must have at least 1 approval from the team of maintainers. At the moment the pool of approvers is only the @aws-powertools/lambda-typescript-core team, however in the past few weeks we have had multiple reviews from members of the @aws-powertools/core team as well.

After discussing with @sthulb, we should add the second team to the pool of reviewer.

Why is this needed?

So that we can merge PRs more quickly when we receive reviews from the core team and require double-reviews.

Which area does this relate to?

Governance

Solution

Add the core team to the CODEOWNERS file.

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

Metadata

Metadata

Assignees

Labels

completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)

Type

No type

Projects

Status

Shipped

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions