pierrejeambrun commented on code in PR #36267:
URL: https://github.com/apache/airflow/pull/36267#discussion_r1429226210
##########
dev/README_RELEASE_PROVIDER_PACKAGES.md:
##########
@@ -1110,6 +1111,8 @@ Make sure you create the following branch from the git
tag (steps before) and no
```shell script
cd ${AIRFLOW_REPO_ROOT}
+git checkout main
+git pull
Review Comment:
I don't think it is of big importance here because we fetch tags (for
airflow versiosn) from remote repository and then do a few things regarding
constraint files.
But I agree that in order to get the latest source for breeze running from
main seems safer I would say. (for core we run from main and let breeze
checkout the branches, so sources are up to date)
##########
dev/README_RELEASE_PROVIDER_PACKAGES.md:
##########
@@ -1110,6 +1111,8 @@ Make sure you create the following branch from the git
tag (steps before) and no
```shell script
cd ${AIRFLOW_REPO_ROOT}
+git checkout main
+git pull
Review Comment:
I don't think it is of big importance here because we fetch tags (for
airflow versiosn) from remote repository and then do a few things regarding
constraint files.
But I agree that in order to get the latest source for breeze running from
main seems safer I would say. (for core we run from main and let breeze
checkout the branches, so sources are up to date for breeze)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
