eladkal commented on a change in pull request #20418:
URL: https://github.com/apache/airflow/pull/20418#discussion_r772195651
##########
File path: tests/providers/amazon/aws/operators/test_s3_bucket.py
##########
@@ -22,7 +22,7 @@
from moto import mock_s3
from airflow.providers.amazon.aws.hooks.s3 import S3Hook
-from airflow.providers.amazon.aws.operators.s3 import S3CreateBucketOperator,
S3DeleteBucketOperator
+from airflow.providers.amazon.aws.operators.s3_bucket import
S3CreateBucketOperator, S3DeleteBucketOperator
Review comment:
Why are you modifying s3 files?
I think you have several unrelated changes of s3 in this PR.
##########
File path: docs/apache-airflow-providers-amazon/operators/sqs_publish.rst
##########
@@ -16,7 +16,7 @@
under the License.
-.. _howto/operator:SQSPublishOperator:
+.. _howto/operator:operator:SQSPublishOperator:
Review comment:
Is this right ? Shouldn't it be?
```suggestion
.. _howto/operator:SqsPublishOperator:
```
##########
File path: docs/apache-airflow-providers-amazon/operators/sqs_publish.rst
##########
@@ -34,7 +34,7 @@ Using Operator
--------------
Use the
-:class:`~airflow.providers.amazon.aws.operators.sqs.SQSPublishOperator`
+:class:`~airflow.providers.amazon.aws.operators.sqs.operator:SQSPublishOperator`
Review comment:
same question
--
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]
