potiuk commented on code in PR #42944:
URL: https://github.com/apache/airflow/pull/42944#discussion_r1797354019
##########
dev/breeze/src/airflow_breeze/utils/python_versions.py:
##########
@@ -45,14 +45,12 @@ def get_python_version_list(python_versions: str) ->
list[str]:
return python_version_list
-def check_python_version():
- error = False
- if not sys.version_info >= (3, 9):
- get_console().print("[error]At least Python 3.9 is required to prepare
reproducible archives.\n")
- error = True
- if error:
+def check_python_version(release_provider_packages: bool = False):
+ if not sys.version_info < (3, 12) and release_provider_packages:
Review Comment:
BTW this will only fal when verifying "apache-beam" provider - so following
the common-sense approach "fail early rather than at the end of process, It
makes absolutely sense to fail it during the check.
Since you rarely (or never) run provider's verification, you should not be
impacted.
--
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]
