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/09/22 10:49:13 UTC

[GitHub] [airflow] potiuk commented on a diff in pull request #26589: Convert release manager instructions to use "partial" svn checkouts

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


##########
dev/README_RELEASE_AIRFLOW.md:
##########
@@ -314,12 +314,12 @@ The Release Candidate artifacts we vote upon should be the exact ones we vote ag
 
     ```shell script
     # First clone the repo
-    svn checkout https://dist.apache.org/repos/dist/dev/airflow airflow-dev
-    cd airflow-dev
-    # Or move into it if you already have it cloned
+
+    [ -d asf-dist ] || svn checkout --depth=immediates https://dist.apache.org/repos/dist asf-dist

Review Comment:
   I like the idea of unifying it iunder arlfow repo. We need to add asf-dist to .gitignore though.



-- 
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