Taragolis commented on code in PR #34784: URL: https://github.com/apache/airflow/pull/34784#discussion_r1347991914
########## airflow/providers/amazon/aws/operators/base_aws.py: ########## Review Comment: And wondering about module naming because I a bit confused: Suggestion in old PR use **base_aws**: https://github.com/apache/airflow/pull/24246#discussion_r892775749 Naming for Base Trigger: airflow.providers.amazon.aws.triggers.**base**.AwsBaseWaiterTrigger cc @ferruzzi @vincbeck @o-nikolas > There are only two hard things in Computer Science: cache invalidation and naming things -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
