-
Notifications
You must be signed in to change notification settings - Fork 2
Revert process
Monica Wheeler edited this page Oct 4, 2021
·
7 revisions
This page is deprecated in favor of the UXD Handbook.
Reverts do need to happen occasionally, and no blame or guilt should result from revert to a merge. We learn as a team and move on.
Read the guidelines on version bump delays before proceeding.
- Assuming recommended processes have been followed, this Pull Request (PR) will be linked from the most recent
sage-lib
version bump PR. If this is not the case, in thesage-lib
GitHub repo, navigate to "Pull Requests" and locate the originating PR to revert in the list of "Closed" PRs. - Scroll to the bottom of the PR to be reverted, and locate the merge description item in the timeline. The description should read "
<username of author>
merged commit<git hash here>
into develop 5 hours ago". - Click the "Revert" button next to the merge description. No need to panic. This will not make any immediate changes to the codebase
- You should now be redirected to a new "Open a pull request" view. By default, the title will read "Revert
<title of PR>
." To demonstrate the reason for this PR, do not alter this title. - Confirm that this PR will is merged into
develop
. - Add a description if necessary, indicating the reason for the revert.
- Scan through the list of changed files, and verify that only the files related to the PR to be reverted are included.
- Assign yourself as the PR assignee.
- Once you have verified all of the information above, click the "Create pull request" to create the PR and assign at minimum one, but preferably two reviewers for testing. If you are not the original author of the feature branch being reverted, be sure to add yourself as a reviewer
- Thoroughly test the branch on your own local machine and confirm that the work in the branch has been reverted and no new issues have appeared.
- Once approval has been given by all reviewers, proceed as you normally would with a feature branch: merge into develop using the "Merge pull request" button, and (optionally) delete the revert branch
- Now that the change has been reverted in
develop
, a new Sage version update is now necessary to merge the revert PR (and any other recent changes) fromdevelop
intomain
.- If a new "Next version bump" PR already exists, navigate to the version bump PR, or
- If a new "Next version bump" PR has yet to be created, navigate to the "Pull requests" tab in GitHub, and click the "New pull request" button, making sure the base branch is set to
main
. Fill out the description using the version bump template
- Follow the remaining steps in the version bump process documentation. Depending on the urgency of the revert, you may be able to skip directly to Step 2.
Thanks for using Sage!
Welcome
Conventions
Processes and Reference