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

[GitHub] [airflow] potiuk commented on pull request #29680: Google Cloud Providers - Introduce GoogleCloudBaseOperator

potiuk commented on PR #29680:
URL: https://github.com/apache/airflow/pull/29680#issuecomment-1439879330

   > LGTM if the tests pass we can merge. That said if you could please add some context at the top. I realize that there is more info in the ticket but it would be nice to have a few sentences here as well to give larger context so future users who are running git blame don't have to dive through a hyperlink rabbithole to find out why this change was made.
   
   yep. Fully agree. Context is important.


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