Skip to content

Commit 8b54316

Browse files
docs: Update CONTRIBUTING.md (#635)
* Update CONTRIBUTING.md * nit: Update CONTRIBUTING.md Co-authored-by: Wassim Chegham <[email protected]> * nit: Update CONTRIBUTING.md Co-authored-by: Wassim Chegham <[email protected]> * nit: Update CONTRIBUTING.md Co-authored-by: Wassim Chegham <[email protected]> Co-authored-by: Wassim Chegham <[email protected]>
1 parent b7a7c27 commit 8b54316

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

CONTRIBUTING.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -361,6 +361,12 @@ Breaking Change section should start with the phrase "BREAKING CHANGE: " followe
361361

362362
Similarly, a Deprecation section should start with "DEPRECATED: " followed by a short description of what is deprecated, a blank line, and a detailed description of the deprecation that also mentions the recommended update path.
363363

364+
### Merging Pull requests (For Code owners)
365+
While merging pull requests, the following guidelines should be followed:
366+
367+
- Before merging, make sure there is at least one approval from the Code-owners.
368+
- While merging the changes, it is advised to "Squash and Merge" instead of "Merge" otherwise all the commits (including chore commits, nit commits, and commits to resolving conflicts) will be shown in the history of the `main` branch of the repo which makes it difficult for anyone who is using the repo to track the commits.
369+
364370
### Revert commits
365371

366372
If the commit reverts a previous commit, it should begin with `revert: `, followed by the header of the reverted commit.

0 commit comments

Comments
 (0)