ferruzzi commented on code in PR #38693:
URL: https://github.com/apache/airflow/pull/38693#discussion_r1550640629


##########
airflow/providers/amazon/aws/waiters/bedrock.json:
##########


Review Comment:
   You know what, I'll admit that the fact that some are combined into 
[test_custom_waiters.py](https://github.com/apache/airflow/blob/main/tests/providers/amazon/aws/waiters/test_custom_waiters.py)
 and others were in their own files messed me up.  
   
![image](https://github.com/apache/airflow/assets/1920178/27699661-ea24-4aeb-a632-f9df4350b770)
   
   I thought we only had tests for Neptune and Databrew because the others are 
"hiding".
   
   I still don't really see the point of those ones, but I'll add them in their 
own file, and we can split the combined file out into one-file-per-service like 
everywhere else so it's more obvious in the future.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to