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 2021/08/18 09:14:05 UTC

[GitHub] [airflow] potiuk commented on a change in pull request #17674: Dev: Remove duplicate step to push Docker Image

potiuk commented on a change in pull request #17674:
URL: https://github.com/apache/airflow/pull/17674#discussion_r691054542



##########
File path: dev/README_RELEASE_AIRFLOW.md
##########
@@ -40,7 +39,7 @@
   - [Publish release to SVN](#publish-release-to-svn)
   - [Prepare PyPI "release" packages](#prepare-pypi-release-packages)
   - [Update CHANGELOG.md](#update-changelogmd)
-  - [Manually prepare production Docker Image](#manually-prepare-production-docker-image-1)
+  - [Manually prepare production Docker Image](#manually-prepare-production-docker-image)

Review comment:
       @kaxil @jhtimmins  The main reason it was twice, because those were two different images (first was RC and the second was final one) :). I see tha tthe RC was prepared and pushed eventually anyway, But at least we should link to the common step rather than remove it from "RC" part as it is good for users to verify (I do it as well) 




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