potiuk commented on pull request #17855: URL: https://github.com/apache/airflow/pull/17855#issuecomment-906732632
I think I addressed all comments. Please take a look @eladkal @leahecole @BasPH @jedcunningham @uranusjr and anyone else interested. The change depends on #17858 - as during testing I found out some issues with pre-commits validating provider.yaml (and #17858 corrects errors resulting from it) To make it easier to review, I squashed all the speling/etc fixes into a single commit, and added all new things in the fixup commit that follows. Those are: * added separate doc-issue * added separate providers issue (with multi-select list of affected providers, yay! - and also added pre-commit to verify if we have all providers there :)) * added nice preamble to explain that maybe you do not need to create the issue at all * added some explanation to versions of airflow (main -> development, 2.1.3 - > latest released version) * better sorting of issues in the issue list (bugs go first) You can try it out here: https://github.com/potiuk/airflow/issues/new/choose -- 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]
