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


##########
dev/README_RELEASE_AIRFLOW.md:
##########
@@ -831,14 +831,24 @@ svn update --set-depth=infinity asf-dist/dev/airflow
 cd asf-dist/dev/airflow/${VERSION_RC}
 for i in *.whl *.tar.gz
 do
-  echo "Checking if $(basename $i) is the same as 
${AIRFLOW_REPO_ROOT}/dist/$(basename $i)"
-  diff "$(basename $i)" "${AIRFLOW_REPO_ROOT}/dist/$(basename $i)" && echo "OK"
+  # This conversion needed for how breeze creates packages and how we release 
them to keep things smooth

Review Comment:
   As discussed in slack - strange you have it. The distributions that are 
stored in SVN (and those that you build locally for verification) - do not have 
RC suffix - currently we do not store rc suffixed distributions in SVN at all.
   
   Probably you did `export VERSION_SUFFIX=rc1` 



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

Reply via email to