o-nikolas commented on code in PR #46405:
URL: https://github.com/apache/airflow/pull/46405#discussion_r1941656216
##########
dev/breeze/src/airflow_breeze/prepare_providers/provider_documentation.py:
##########
@@ -1222,6 +1222,13 @@ def _regenerate_pyproject_toml(context: dict[str, Any],
provider_details: Provid
context["EXTRAS_REQUIREMENTS"] = "\n".join(optional_dependencies)
context["DEPENDENCY_GROUPS"] = "\n".join(dependency_groups)
+ # Most providers require the same python versions, but some may have
exclusions
+ requires_python_version = "~=3.9"
+ for excluded_python_version in provider_details.excluded_python_versions:
+ requires_python_version += f", !={excluded_python_version}"
Review Comment:
Ah yes, I remember now, ruff fails when I remove the space. It has a hard
time parsing the version specifier when it's omitted:
```
ruff failed
Cause: Failed to parse
/home/ANT.AMAZON.COM/onikolas/code/oss/airflow/team_fork/providers/cloudant/pyproject.toml
Cause: TOML parse error at line 52, column 19
|
52 | requires-python = "3.9,!=3.9"
| ^^^^^^^^^^^
Failed to parse version: Unexpected end of version specifier, expected
operator:
3.9,!=3.9
^^^
```
--
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]
