You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/02 20:49:42 UTC

[GitHub] [airflow] josh-fell edited a comment on pull request #19956: Clean up incorrect class names of Google system tests

josh-fell edited a comment on pull request #19956:
URL: https://github.com/apache/airflow/pull/19956#issuecomment-984745350


   > What’s the criteria you’re using to decide whether to prefix a class with `GCP` or not?
   
   That portion of the class names I didn't change. I did take a little liberty using "GCS" instead of "GoogleCloudStorage" if the class name was getting wildly long and unreadable. There didn't seem to be a clear naming convention that was followed. I just focused on making sure the class name at least contained the name of related service based on the file name and example DAG.


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