potiuk commented on PR #27829: URL: https://github.com/apache/airflow/pull/27829#issuecomment-1386902562
> Hi @potiuk, It seems like there's something wrong with the breeze build on this PR. For example, here: https://github.com/apache/airflow/actions/runs/3947115412/jobs/6755756155#step:8:33 it's saying that the command does not exist. I think the previous command cleaned up something or added something (egg.info?) that confused installed breeze. Try to reproduce it and run in the same sequence and see if you can repeat it. Also maybe the "hidden=True" confused it additionally - unhide it and see if it helps * -rc and -prev-r are really wrongly defined at least in the way I understand "short" options. They should not be just "shorter". Single '-' indicates that the option is ONE_LETTER only - this is in order to allow to combine several flags together. So the `-rc` is really equivalent of `-r` and `-c` and those options. Maybe that also confuses click. BTW. There is absolutely no reason to use short (or even shorter) options in CI. They only remove readability (longer options are more readable and give more context by just reading them - so I always try to - at least - use longer options in CI. > Also, I have been having issues with post-breeze step This is the same reason as above - there is a "git clean" and likely other steps that are used in the release process and they are in order to make sure that there are no accidental artifacts addedd while preparing the release. But those (in this case checked out submodules) are needed by the CI - so the clean command should be skipped on CI -- 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]
