You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "uranusjr (via GitHub)" <gi...@apache.org> on 2023/06/05 04:16:29 UTC
[GitHub] [airflow] uranusjr opened a new pull request, #31708: Microsoft provider docstring improvements
uranusjr opened a new pull request, #31708:
URL: https://github.com/apache/airflow/pull/31708
Related suggestions found when reviewing #31425.
--
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
[GitHub] [airflow] vincbeck commented on a diff in pull request #31708: Microsoft provider docstring improvements
Posted by "vincbeck (via GitHub)" <gi...@apache.org>.
vincbeck commented on code in PR #31708:
URL: https://github.com/apache/airflow/pull/31708#discussion_r1218536745
##########
airflow/providers/microsoft/azure/hooks/data_lake.py:
##########
@@ -214,8 +210,7 @@ def download_file(
)
def list(self, path: str) -> list:
- """
- List files in Azure Data Lake Storage.
+ """List files in Azure Data Lake Storage
Review Comment:
```suggestion
"""List files in Azure Data Lake Storage.
```
##########
airflow/providers/microsoft/azure/hooks/data_lake.py:
##########
@@ -225,8 +220,7 @@ def list(self, path: str) -> list:
return self.get_conn().walk(path)
def remove(self, path: str, recursive: bool = False, ignore_not_found: bool = True) -> None:
- """
- Remove files in Azure Data Lake Storage.
+ """Remove files in Azure Data Lake Storage
Review Comment:
```suggestion
"""Remove files in Azure Data Lake Storage.
```
--
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
[GitHub] [airflow] ephraimbuddy merged pull request #31708: Microsoft provider docstring improvements
Posted by "ephraimbuddy (via GitHub)" <gi...@apache.org>.
ephraimbuddy merged PR #31708:
URL: https://github.com/apache/airflow/pull/31708
--
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