OmerJog commented on issue #6713: Add some r/w functionality to GCP Bigtable Hook URL: https://github.com/apache/airflow/pull/6713#issuecomment-566530923 @EmilyMazo can you please fix errors ``` airflow/gcp/hooks/bigtable.py:260:111: E501 line too long (138 > 110 characters) airflow/gcp/hooks/bigtable.py:270:111: E501 line too long (120 > 110 characters) airflow/gcp/hooks/bigtable.py:277:111: E501 line too long (137 > 110 characters) airflow/gcp/hooks/bigtable.py:288:111: E501 line too long (120 > 110 characters) airflow/gcp/hooks/bigtable.py:315:16: F821 undefined name 'ValueRegexFilter' airflow/gcp/hooks/bigtable.py:318:5: E303 too many blank lines (2) airflow/gcp/hooks/bigtable.py:318:111: E501 line too long (112 > 110 characters) airflow/gcp/hooks/bigtable.py:328:111: E501 line too long (120 > 110 characters) airflow/gcp/hooks/bigtable.py:337:111: E501 line too long (213 > 110 characters) airflow/gcp/hooks/bigtable.py:337:129: F821 undefined name 'RowFilter' airflow/gcp/hooks/bigtable.py:339:111: E501 line too long (111 > 110 characters) airflow/gcp/hooks/bigtable.py:356:111: E501 line too long (113 > 110 characters) airflow/gcp/hooks/bigtable.py:360:111: E501 line too long (120 > 110 characters) tests/gcp/hooks/test_bigtable.py:42:1: E302 expected 2 blank lines, found 1 tests/gcp/hooks/test_bigtable.py:482:111: E501 line too long (148 > 110 characters) tests/gcp/hooks/test_bigtable.py:482:141: E999 SyntaxError: positional argument follows keyword argument ``` Also note that the Jira in your commit message is wrong. It should be `AIRFLOW-6155` not `AIRFLOW-6713`
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
