ashb commented on a change in pull request #8898:
URL: https://github.com/apache/airflow/pull/8898#discussion_r427240664
##########
File path: breeze
##########
@@ -1629,9 +1647,13 @@ function flag_push_docker_images() {
# Prints flags that control version of generated packages
function flag_version_suffix() {
echo "
--S, --version-suffix
- Adds optional suffix to the generated backport package version. It can
be used to generate
- rc1/rc2 ... versions of the packages.
+-S, --version-suffix-for-pypi
+ Adds optional suffix to the version in the generated backport package.
It can be used
+ to generate rc1/rc2 ... versions of the packages to be uploaded to
PyPI.
Review comment:
> I think historically this was done so that even if someone installed
the rc packages when they do `airflow version` it shows `rc` string in it.
However maybe for backport packages it might not matter, thoughts @ashb ?
Yeah, since Airflow has `airflow.__version__` we want that to be right for
RC releases, but as you say, the backport providers don't have this attribute
anywhere (and it's probably not needed to add one) so this is a good approach
for providers.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
