Skip to content

Commit 3245314

Browse files
authored
Update securing-repositories.md
1 parent e4db996 commit 3245314

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

practices/securing-repositories.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,8 @@ This minimum set of requirements should be implemented alongside other relevant
5858
- Code reviews must be approved by at least one code owner.
5959
- You may want to require multiple code owners to review pull requests.
6060
- Commits must be <!-- markdown-link-check-disable -->[signed](https://docs.github.com/en/github/administering-a-repository/defining-the-mergeability-of-pull-requests/about-protected-branches#require-signed-commits)<!-- markdown-link-check-enable --> and verified before merging.
61-
- Git treats authentication and identity separately - any authenticated user can impersonate another developer when committing code. This means that even if a junior account is compromised it could have significant consequences, for example impersonating the lead developer in the hope of an easy merge. Only by requiring signing can identity truly be verified. [Setup Guides](guides/commit-signing.md) for macOS, Windows, GitHub Actions, and AWS CodePipeline.
61+
- Git treats authentication and identity separately - any authenticated user can impersonate another developer when committing code. This means that even if a junior account is compromised it could have significant consequences, for example impersonating the lead developer in the hope of an easy merge. Only by requiring signing can identity truly be verified.
62+
- For further details, please see [Setup Guides](guides/commit-signing.md) for macOS, Windows, GitHub Actions, and AWS CodePipeline.
6263
- Existing reviews must be invalidated automatically when new commits are pushed (using the `fresh-commits-invalidate-existing-reviews` option).
6364
- Merging must be blocked if the branch is not up to date.
6465
- Consider any further automated status checks which should be enforced prior to merging a branch

0 commit comments

Comments
 (0)