-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-6413] Add config file for Mergeable Github integration #6981
Conversation
16f312f
to
8dd300e
Compare
8dd300e
to
4132612
Compare
(cherry picked from commit 991b34d)
begins_with: | ||
match: '[AIRFLOW-' | ||
must_include: | ||
regex: ^(\[AIRFLOW-XXXX\]|\[AIRFLOW\-\d{1,4}\]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be AIRFLOW-XXX
not XXXX
:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:D This was intentional as there was a recent change in Github PR template that asked users to add 4 X's [AIRFLOW-XXXX]
for docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That reminds you that if you have XXX in the release scripts to distinguish doc-only updates, it should be updated to support also XXXX (maybe - depends if you have in search pattern "]" ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW. I really love that people will have to manually click the checkboxes now (though initially people won't understand why mergable complains). Happened already with @michalslowikowski00 :D. But I think this is good if they ask.
Add config for https://github.com/apps/mergeable
Jira: https://issues.apache.org/jira/browse/AIRFLOW-6413
[AIRFLOW-XXXX]
for document-only changesIn case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.