turbaszek commented on a change in pull request #14191:
URL: https://github.com/apache/airflow/pull/14191#discussion_r574632601
##########
File path: airflow/providers/google/suite/hooks/drive.py
##########
@@ -218,3 +218,9 @@ def upload_file(self, local_location: str, remote_location:
str) -> str:
)
self.log.info("File %s uploaded to gdrive://%s.", local_location,
remote_location)
return file.get("id")
+
+ def download_file(self, file_id: str, file_handle):
Review comment:
A docstring is required for every public function, otherwise pylint will
fail on CI. I recommend installing pre-commits to check it locally:
https://github.com/apache/airflow/blob/master/STATIC_CODE_CHECKS.rst#prerequisites-for-pre-commit-hooks
----------------------------------------------------------------
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:
[email protected]
