You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/05/26 15:53:16 UTC

[GitHub] [airflow] potiuk commented on a diff in pull request #23928: Clarify manual merging of PR in release doc

potiuk commented on code in PR #23928:
URL: https://github.com/apache/airflow/pull/23928#discussion_r882824735


##########
dev/README_RELEASE_AIRFLOW.md:
##########
@@ -224,7 +224,12 @@ The Release Candidate artifacts we vote upon should be the exact ones we vote ag
 
 - Update the `REVISION_HEADS_MAP` at airflow/utils/db.py to include the revision head of the release even if there are no migrations.
 - Commit the version change.
-- PR from the 'test' branch to the 'stable' branch, and manually merge it once approved.
+- PR from the 'test' branch to the 'stable' branch, and manually merge it once approved. Here's how to manually merge the PR:

Review Comment:
   I think it should be after checking out stable actually @ephraimbuddy 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org