o-nikolas commented on code in PR #33960:
URL: https://github.com/apache/airflow/pull/33960#discussion_r1312320477
##########
airflow/providers/amazon/provider.yaml:
##########
@@ -710,3 +710,16 @@ additional-extras:
- name: cncf.kubernetes
dependencies:
- apache-airflow-providers-cncf-kubernetes>=7.2.0
+
+config:
+ aws:
+ description: This section applies settings for Amazon Web Services (AWS)
integration.
+ options:
+ session_factory:
+ description: |
+ Full import path to the class which implements custom session
factory for ``boto3.session.Session``,
+ for more detail please have a look at
:ref:`howto/connection:aws:session-factory`.
Review Comment:
Just a minor nit:
```suggestion
Full import path to the class which implements a custom session
factory for ``boto3.session.Session``.
For more details please have a look at
:ref:`howto/connection:aws:session-factory`.
```
##########
airflow/providers/amazon/provider.yaml:
##########
@@ -710,3 +710,16 @@ additional-extras:
- name: cncf.kubernetes
dependencies:
- apache-airflow-providers-cncf-kubernetes>=7.2.0
+
+config:
+ aws:
+ description: This section applies settings for Amazon Web Services (AWS)
integration.
Review Comment:
Maybe contains instead of applies?
```suggestion
description: This section contains settings for Amazon Web Services
(AWS) integration.
```
--
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]
