expressjs / expressjs/discussions
EFI: Codeowners
- Dominant language
- No language data
- Stars
- 73
- Forks
- 25
- PR merge metrics
- No merged PRs in 30d
Description
### Motivation
Automation can only be effective if setup correctly. Relying on Codeowners can great help the automation allowing us to automatically notify specific people when a Pull Request is opened on specific part of the project.
It can also be used to notify specific team depending on the change, for example a security group or technical writer if this only change some documentation
### Expectation
Implement Codeowners files in each repositories and setup them accordingly
### Implementation
Setup codeowner
### Status
Part: Organization
### Draft
> In a project, all changes are not equal and the risk is not the same. Contributions to the core package need to be reviewed extensively to ensure the highest quality but other parts are less critical. By using codeowners file, we can automatically manage this risk and allocate specific teams to review pull requests.
For example, a change only targeting documentation or test could be reviewed and validated by the triager team (or another specific team) and does need less approval (maybe only 1-2) while changes in the core of the package may require more validations (3-4 approvals by specific members)
Contributor guide
Assessment
This issue has not been assessed yet.