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 2021/01/05 12:26:51 UTC

[GitHub] [airflow] ashb edited a comment on pull request #13440: Additional properties should be allowed in provider schema

ashb edited a comment on pull request #13440:
URL: https://github.com/apache/airflow/pull/13440#issuecomment-754585936


   Thinking about this more, I'm not sure we even need the "doc build only" information in these files -- we have `docs/apache-airflow-providers-$x/` where we could store this information.
   
   Or it could live in the airflow-site repo. My thinking there is that it is only used for building docs, so having to make extra commits in the apache/airflow monorepo seems un-necessary given airflow-site is what needs that information.
   
   Happy to make these changes myself if you both agree with this approach @mik-laj @potiuk.
   
   In summary:
   
   - Have providers.yaml file only be what is needed at runtime for Airflow
   - Move the information used to build the docs/site to somewhere in to the airflow-site repo (versions, logo, etc)


----------------------------------------------------------------
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