potiuk commented on code in PR #35586:
URL: https://github.com/apache/airflow/pull/35586#discussion_r1393116341
##########
dev/breeze/src/airflow_breeze/commands/release_management_commands.py:
##########
@@ -219,11 +226,24 @@ def prepare_airflow_packages(
sys.exit(result_command.returncode)
+def provider_documentation_summary(documentation: str, message_type:
MessageType, packages: list[str]):
+ if packages:
+ get_console().print(f"{documentation}: {len(packages)}\n")
+ get_console().print(f"[{message_type.value}]{' '.join(packages)}")
+ get_console().print()
+
+
@release_management.command(
name="prepare-provider-documentation",
help="Prepare CHANGELOG, README and COMMITS information for providers.",
)
-@option_debug_release_management
+@click.option(
+ "--skip-git-fetch",
+ is_flag=True,
+ help="Skips removal and recreation of `apache-https-rof-providers` remote
in git. By default, the "
Review Comment:
Yeah, It's massive, but I was not able to do it gradually :(. I am doing
"command-by-command" on that one - so this one has some remnants that are
removed only in the follow-up one (#35617 - and only there you see some of the
code finally removed)
Really sorry for that, but I found no other way to do it :( (so overall
those 2 PRS combined actually remove(!) 300 lines of code - this one's +1300 is
just an intermediate step where a lot of duplication remains).
It's mostly really moving code from one place to the other while cleaning
necessary bits to make the moved code fit in the new place.
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
