Skip to content

Commit ccb2da2

Browse files
committed
Move update branch section after submitting
1 parent b89ba85 commit ccb2da2

1 file changed

Lines changed: 16 additions & 16 deletions

File tree

‎getting-started/pull-request-lifecycle.rst‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -526,6 +526,22 @@ another so they can easily verify whether their comments have been addressed.
526526
The commits will be squashed when the pull request is merged.
527527

528528

529+
:guilabel:`Update branch` button
530+
================================
531+
532+
You can click on the :guilabel:`Update branch` button to merge the latest
533+
changes from the base branch (usually ``main``) into the PR.
534+
This is useful to :ref:`keep the CI green <keeping-ci-green>` for old PRs,
535+
or to check if a CI failure has been fixed in the base branch.
536+
537+
If the PR is very old, it may be useful to update the branch before merging to
538+
ensure that the PR does not fail any CI checks that were added or changed since
539+
CI last ran.
540+
541+
Do not click :guilabel:`Update branch` without a good reason because it notifies
542+
everyone watching the PR that there are new changes, when there are not,
543+
and it uses up limited CI resources.
544+
529545
Converting an existing patch from b.p.o to GitHub
530546
=================================================
531547

@@ -685,22 +701,6 @@ If you identify such flaky behavior, look for an issue in the `issue tracker`_
685701
that describes this particular flakiness. Create a new issue if you can't
686702
find one.
687703

688-
:guilabel:`Update branch` button
689-
================================
690-
691-
You can click on the :guilabel:`Update branch` button to merge the latest
692-
changes from the base branch (usually ``main``) into the PR.
693-
This is useful to :ref:`keep the CI green <keeping-ci-green>` for old PRs,
694-
or to check if a CI failure has been fixed in the base branch.
695-
696-
If the PR is very old, it may be useful to update the branch before merging to
697-
ensure that the PR does not fail any CI checks that were added or changed since
698-
CI last ran.
699-
700-
Do not click :guilabel:`Update branch` without a good reason because it notifies
701-
everyone watching the PR that there are new changes, when there are not,
702-
and it uses up limited CI resources.
703-
704704
Committing/rejecting
705705
====================
706706

0 commit comments

Comments
 (0)