mirror of
https://github.com/mongodb/mongo.git
synced 2024-11-21 12:39:08 +01:00
SERVER-91397 Improve PR template and override criteria (#23334)
GitOrigin-RevId: 18f78af685dc56c2bbf0f08a5d4ed96cbf4467e7
This commit is contained in:
parent
a6f71fcee6
commit
7723ea82c4
2
.github/PULL_REQUEST_TEMPLATE/README.md
vendored
2
.github/PULL_REQUEST_TEMPLATE/README.md
vendored
@ -6,6 +6,8 @@ These custom templates can be used for example, by individual teams to have a cu
|
||||
|
||||
Read more in [Github's docs](https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository)
|
||||
|
||||
If you update the default PR template, you also need to update the commit metadata in github branch rulesets.
|
||||
|
||||
# How To Use This Folder
|
||||
|
||||
To create a custom template, create a new markdown file in this folder.
|
||||
|
21
.github/pull_request_template.md
vendored
21
.github/pull_request_template.md
vendored
@ -1,20 +1 @@
|
||||
# Issue
|
||||
|
||||
<!-- Jira issue key and link. Entering a SERVER issue will automatically create a link.-->
|
||||
|
||||
# Description
|
||||
|
||||
<!-- Please include a summary of the changes and the related issue. Be sure to include relevant motivation and context. -->
|
||||
|
||||
# Testing
|
||||
|
||||
- [ ] <!-- Please describe the tests you ran to verify your changes and include any links to patches. Consider using `evergreen patch` described below.-->
|
||||
|
||||
<!--
|
||||
Available Evergreen Comment Commands
|
||||
evergreen patch - attempts to create a new PR patch that reruns on each new commit; this is required to create a PR patch when only manual PR testing is enabled
|
||||
evergreen keep-definitions - reuse the tasks from the previous patch in subsequent patches. Use this after you modify a patch's tasks.
|
||||
evergreen reset-definitions - reset the patch tasks to the original definition
|
||||
evergreen refresh - resyncs PR GitHub checks
|
||||
evergreen help - comments the above info
|
||||
-->
|
||||
Anything in this description will be included in the commit message. Replace or delete this text before merging. Add links to testing in the comments of the PR.
|
||||
|
@ -1,7 +1,5 @@
|
||||
# Merging
|
||||
|
||||
## Merging into master
|
||||
|
||||
### Standard Merge
|
||||
|
||||
If you have some changes locally that you want to merge into the mongodb repo, you must create a [pull request](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request?tool=webui#creating-the-pull-request). Once the pull request is created you must get approval from an [owner](owners_format.md) of the files changed. If the files changed have no owner then you must get approval from one other engineer. Once you have gotten approval you will see a green merge button which, when pressed, will merge your code.
|
||||
@ -14,9 +12,9 @@ If you have changes that fall into one of the following categories you can get a
|
||||
2. You are reverting a change that has caused a failure in mainline.
|
||||
3. You are making a large scale change. For example, renaming a prolific typo in the codebase, applying formatting, applying a generated fix, etc...
|
||||
|
||||
To get an override you must get approval from any module owner or a member of 10gen/mongo-break-glass (module owners should be members of 10gen/mongo-break-glass).
|
||||
To get an override you must get approval from any module owner or a member of [10gen/mongo-break-glass](https://mana.corp.mongodbgov.com/resources/664e9bed3d7d150379d3e0d0) (module owners should be members of 10gen/mongo-break-glass).
|
||||
|
||||
Once you have the approvals needed, the person providing the override must wait to a merge queue patch pass with the changes and a base merged within 8 hours (should be verified by the overrider). The overrider will then click the checkbox and the red "Merge pull request" button.![override_image](override_image.png)
|
||||
Once you have the approvals needed, the person providing the override must wait for a merge-queue patch to complete without failures and a base merged within 8 hours (should be verified by the overrider). The overrider will then click the checkbox and the red "Merge pull request" button.![override_image](override_image.png)
|
||||
This will skip the merge queue and force all merges in the queue to restart since the base was updated.
|
||||
|
||||
## Merging into an older branch (backports)
|
||||
|
Loading…
Reference in New Issue
Block a user