uranusjr commented on code in PR #32854:
URL: https://github.com/apache/airflow/pull/32854#discussion_r1274544726
##########
dev/provider_packages/prepare_provider_packages.py:
##########
@@ -1676,10 +1676,12 @@ def verify_changelog_exists(package: str) -> str:
def list_providers_packages():
"""List all provider packages."""
providers = get_all_providers()
- # For now we should exclude open-lineage from being consider for releasing
until it is ready to
- # be released
- if "openlineage" in providers:
- providers.remove("openlineage")
+ # if provider needs to be not considered in release add it here
+ # this is useful for cases where provider is WIP for a long period thus we
don't want to release it yet.
+ providers_to_remove_from_release = []
+ for provider in providers_to_remove_from_release:
+ if provider in providers:
+ providers.remove(provider)
for provider in providers:
console.print(provider)
Review Comment:
Why do the in-place modification? We can just stick a `if provider in
providers_to_remove_from_release: continue` in the latter for loop instead.
--
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]
