You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/10/27 09:03:47 UTC

[GitHub] [airflow] mik-laj edited a comment on issue #6439: [AIRFLOW-5733] Migrate AWS components to /providers/aws [AIP-21]

mik-laj edited a comment on issue #6439: [AIRFLOW-5733] Migrate AWS components to /providers/aws [AIP-21]
URL: https://github.com/apache/airflow/pull/6439#issuecomment-546675011
 
 
   > Excluded full directory '_api/airflow/providers/aws' from the docs for now, a number of components are missing docs which should be handled in a different PR.
   
   I think it is necessary to create a separate PR that will create this package and then make changes to the [https://github.com/apache/airflow/blob/master/docs/autoapi_templates/index.rst](docs/autoapi_templates/index.rst]) file so that the documentation builds correctly.
   
   > Components are now grouped by AWS service (operators/s3.py became quite long)
   
   Can you do it in a separate PR so that it is possible to verify that it was done correctly? One PR can introduce tests, and then another PR can check if the path is still working properly.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services