Copilot commented on code in PR #64554: URL: https://github.com/apache/airflow/pull/64554#discussion_r3025327180
########## pyproject.toml: ########## @@ -1347,7 +1347,6 @@ required-version = ">=0.9.17" # Synchroonize with scripts/ci/prek/upgrade_important_versions.py exclude-newer = "4 days" no-build-isolation-package = ["sphinx-redoc"] -constraint-dependencies = ["lxml==6.0.2"] # Remove after https://github.com/aws/amazon-redshift-python-driver/pull/272 [tool.uv.sources] Review Comment: PR description says the lxml workaround can be removed now that redshift-connector 2.1.13 is released, but the repo is still locking/redshift dependency below that (uv.lock currently resolves redshift-connector==2.1.7, and providers/amazon/pyproject.toml still allows redshift_connector>=2.1.3). Without bumping the minimum/lock to 2.1.13+, removing this constraint may be premature if older redshift-connector versions still require the workaround. Suggest bumping the provider dependency to >=2.1.13 (and regenerating uv.lock) or clarifying why the workaround is safe to remove without upgrading. -- 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]
