o-nikolas commented on code in PR #47321:
URL: https://github.com/apache/airflow/pull/47321#discussion_r1982194074
##########
providers/amazon/src/airflow/providers/amazon/aws/operators/s3.py:
##########
@@ -51,38 +52,40 @@ class S3CreateBucketOperator(BaseOperator):
:param bucket_name: This is bucket name you want to create
:param aws_conn_id: The Airflow connection used for AWS credentials.
- If this is None or empty then the default boto3 behaviour is used. If
+ If this is ``None`` or empty then the default boto3 behaviour is used.
If
running Airflow in a distributed manner and aws_conn_id is None or
empty, then default boto3 configuration would be used (and must be
maintained on each worker node).
- :param region_name: AWS region_name. If not specified fetched from
connection.
+ :param region_name: AWS region_name. If not specified then the default
boto3 behaviour is used.
+ :param verify: Whether or not to verify SSL certificates. See:
+
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/core/session.html
+ :param botocore_config: Configuration dictionary (key-values) for botocore
client. See:
+
https://botocore.amazonaws.com/v1/documentation/api/latest/reference/config.html
"""
- template_fields: Sequence[str] = ("bucket_name",)
+ template_fields: Sequence[str] = aws_template_fields(
+ "bucket_name",
+ )
Review Comment:
Yeah, that's a good call. The comma was there before to ensure that was a
tuple, but now that it's a function call it's not needed. I'll update it!
--
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]
