You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Taragolis (via GitHub)" <gi...@apache.org> on 2023/02/18 21:33:03 UTC

[GitHub] [airflow] Taragolis commented on a diff in pull request #29549: Fix and augment `check-for-inclusive-language` CI check

Taragolis commented on code in PR #29549:
URL: https://github.com/apache/airflow/pull/29549#discussion_r1111114409


##########
airflow/providers/google/cloud/hooks/bigquery.py:
##########
@@ -240,7 +240,7 @@ def get_pandas_df(
         query. The DbApiHook method must be overridden because Pandas
         doesn't support PEP 249 connections, except for SQLite. See:
 
-        https://github.com/pydata/pandas/blob/master/pandas/io/sql.py#L447
+        https://github.com/pydata/pandas/blob/main/pandas/io/sql.py#L447

Review Comment:
   ```suggestion
   ```
   
   I think it could be removed, if we attach link to the branch without specific commit then reference to line(s) could be changes thought the time:
   
   Right not it refers to
   ```console
           Whether to use nullable dtypes as default when reading data. If
   ```
   
   



-- 
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