Taragolis commented on code in PR #37862:
URL: https://github.com/apache/airflow/pull/37862#discussion_r1510230181
##########
airflow/providers/google/cloud/hooks/bigquery.py:
##########
@@ -49,7 +49,7 @@
from google.cloud.exceptions import NotFound
from googleapiclient.discovery import Resource, build
from pandas_gbq import read_gbq
-from pandas_gbq.gbq import GbqConnector # noqa
+from pandas_gbq.gbq import GbqConnector # noqa: F401 Used in
``airflow.contrib.hooks.bigquery``
Review Comment:
🤔
##########
airflow/providers/snowflake/operators/snowflake.py:
##########
@@ -449,7 +449,7 @@ class SnowflakeSqlApiOperator(SQLExecuteQueryOperator):
When executing the statement, Snowflake replaces placeholders (?
and :name) in
the statement with these specified values.
:param deferrable: Run operator in the deferrable mode.
- """ # noqa
+ """ # noqa: D205, D400
Review Comment:
I guess originally it complains about too long line in docstring, however
not it doesn't violent if formatters / linter but we hide accidentally:
- `D205`
[blank-line-after-summary](https://docs.astral.sh/ruff/rules/blank-line-after-summary/)
- `D400`
[ends-in-period](https://docs.astral.sh/ruff/rules/ends-in-period/)
--
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]
