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 2020/01/03 17:48:16 UTC
[GitHub] [airflow] potiuk commented on a change in pull request #7015:
[AIRFLOW-6436] Create & Automate docs on Airflow Configs
potiuk commented on a change in pull request #7015: [AIRFLOW-6436] Create & Automate docs on Airflow Configs
URL: https://github.com/apache/airflow/pull/7015#discussion_r362899422
##########
File path: .github/mergeable.yml
##########
@@ -50,3 +50,8 @@ mergeable:
changed:
file: 'package.json'
files: ['yarn.lock']
+ # If config.yml is updated, so should default_airflow.cfg
+ - do: dependent
Review comment:
I was too late (just started to write this comment) I think this might be not good check. If someone modifies default_airflow.cfg and changes value or add space, probot will complain. I guess pre-commit-config should be more than enough.
----------------------------------------------------------------
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