You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/09/12 12:06:38 UTC

[GitHub] [airflow] ashb commented on a diff in pull request #26328: Convert google provider and its tests to __future__.annotations

ashb commented on code in PR #26328:
URL: https://github.com/apache/airflow/pull/26328#discussion_r968330415


##########
.pre-commit-config.yaml:
##########
@@ -213,7 +213,9 @@ repos:
       - id: pyupgrade
         name: Upgrade Python code automatically
         args: ["--py37-plus"]
-        exclude: ^airflow/_vendor/
+        # We need to exclude gcs hook from pyupgrade because it has public "list" command which clashes
+        # with `list` that is used as type

Review Comment:
   Is this a bug in pyupgrade?



-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org