Skip to content

Commit a0b078b

Browse files
authored
Update CONTRIBUTING.md
This commit adds a section on the repo's branch protection rules to the CONTRIBUTING doc.
1 parent be8072f commit a0b078b

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

CONTRIBUTING.md

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ To suggest a feature or enhancement, open an issue on GitHub with the label
3030
`feature` or `enhancement` using the available feature request issue template.
3131
Please ensure the feature or enhancement has not already been suggested.
3232

33-
## Open a discussion
33+
## Open a Discussion
3434

3535
If you want to start a conversation with the community and maintainers,
3636
we encourage you to use
@@ -48,7 +48,7 @@ To contribute to F5 NGINX documentation, follow these steps:
4848
Alternatively, you're welcome to suggest improvements to highlight problems with
4949
our documentation as described in our [support](./SUPPORT.md) page.
5050

51-
### Git Style Guide
51+
### Git style guide
5252

5353
- Keep a clean, concise and meaningful Git commit history on your branch, rebasing locally and squashing before you submit a PR
5454
- Follow the guidelines of writing a good commit message as described here <https://chris.beams.io/posts/git-commit/>
@@ -61,6 +61,15 @@ our documentation as described in our [support](./SUPPORT.md) page.
6161
- Add more detailed description in the body of the git message (`git commit -a` to give you more space and time in
6262
your text editor to write a good message instead of `git commit -am`)
6363

64+
#### Branch protection rules
65+
66+
This repository has the following branch protection rules in place:
67+
68+
- **Pushing branches that contain the "internal/" prefix is not allowed.** This ensures internal development branches are not accidentally or purposefully pushed to this repo.
69+
- **Two approvers are required for all merges to main and release branches.** This ensures all code that is approved for release to production is appropriately reviewed. This rule applies to all branches with `*release*` in the branch name.
70+
- **Only NGINX DocOps Team members can create release branches.** This ensures the docs team is aware of all branches supporting specific product releases. This rule applies to all branches with `*release*` in the branch name.
71+
- **Pushes (force or otherwise) directly to main or release branches is not allowed.** Release branches serve as "main" for the release they are associated with. Restricting pushes directly to main and release branches ensures all content changes are reviewed and approved. This rule applies to all branches with `*release*` in the branch name and to "main".
72+
6473
### Documentation style guide
6574

6675
For detailed guidance, see our documentation [style guide](./templates/style-guide.md).

0 commit comments

Comments
 (0)