Skip to content

Comments

Remove JIRA requirements from PRs#7771

Merged
kaxil merged 3 commits intoapache:masterfrom
dimberman:AIRFLOW-7094_github-actions
Mar 19, 2020
Merged

Remove JIRA requirements from PRs#7771
kaxil merged 3 commits intoapache:masterfrom
dimberman:AIRFLOW-7094_github-actions

Conversation

@dimberman
Copy link
Contributor

@dimberman dimberman commented Mar 19, 2020

As a part of our move from JIRA tickets to github issues, we should no
longer force users to add JIRA ticket numbers to the titles of their PRs


Issue link: WILL BE INSERTED BY boring-cyborg

Make sure to mark the boxes below before creating PR: [x]


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

As a part of our move from JIRA tickets to github issues, we should no
longer force users to add JIRA ticket numbers to the titles of their PRs
@dimberman dimberman force-pushed the AIRFLOW-7094_github-actions branch from 6b34ebd to 2a9664f Compare March 19, 2020 20:59
@dimberman dimberman requested review from kaxil, potiuk and turbaszek and removed request for turbaszek March 19, 2020 20:59
matchers:
jiraIssueMatch:
titleIssueIdRegexp: \[(AIRFLOW-[0-9]{4})\]
descriptionIssueLink: "[${1}](https://issues.apache.org/jira/browse/${1})"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kaxil is it safe to leave this block in for our transition period where people still might be targeting Jiras?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it is safe. If it doesn't find the Jira in the title it won't do anything. However we need to remove verifyTitles block

@kaxil kaxil merged commit 92ca751 into apache:master Mar 19, 2020
@dimberman dimberman deleted the AIRFLOW-7094_github-actions branch March 19, 2020 21:36
ashb added a commit to ashb/airflow that referenced this pull request Mar 20, 2020
Follow up to apache#7771, where we missed one final check
kaxil pushed a commit that referenced this pull request Mar 20, 2020
Follow up to #7771, where we missed one final check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants