jedcunningham commented on code in PR #51815:
URL: https://github.com/apache/airflow/pull/51815#discussion_r2152417320
##########
dev/breeze/src/airflow_breeze/commands/release_management_commands.py:
##########
@@ -2675,11 +2675,13 @@ def print_issue_content(
"--previous-release",
type=str,
help="commit reference (for example hash or tag) of the previous release.",
+ required=True,
Review Comment:
Turns out the latest flag (TIL) doesn't work in the context of a release -
the tag it tries to use for the latest release is wrong.
Semi-related: flags not used in the release process (or normal related
activities) add complexity - this has _never_ worked in the context of actually
generating the issue contents during a chart release. I realize it was added to
make CI a bit more robust, but I think we can do that in a more static way more
easily.
I'll probably end up opening a follow up PR to refactor the latest flag
away. I'll bet you and CI are the only ones that have ever used it :)
--
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]
