Skip to content

Clarify that backport PR needs to include GH- from master #392

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

Merged
merged 1 commit into from
Jul 21, 2018

Conversation

Mariatta
Copy link
Member

Backport PR needs to be in the form of [X.Y] title (GH-NNNN), this is the pattern that bedevere uses:

https://github.com/python/bedevere/blob/master/bedevere/backport.py#L11

Adding a bookmark link, so I can reference it from a new status check proposed in python/core-workflow#211 (comment)

Mariatta added a commit to python/bedevere that referenced this pull request Jun 24, 2018
If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111
Copy link
Collaborator

@willingc willingc left a comment

Choose a reason for hiding this comment

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

@Mariatta Looks good. Would you mind rebasing and merging? Thanks 👍

@Mariatta Mariatta force-pushed the clarify-backport-pr-title branch from bedd994 to 6236a1e Compare July 13, 2018 18:04
@Mariatta Mariatta merged commit 2ae9f78 into master Jul 21, 2018
@Mariatta Mariatta deleted the clarify-backport-pr-title branch July 21, 2018 02:08
@Mariatta
Copy link
Member Author

Thanks!

Mariatta added a commit to python/bedevere that referenced this pull request Jul 21, 2018
If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111
Mariatta added a commit to python/bedevere that referenced this pull request Jul 21, 2018
* Post a status check for backport PR title.

If a PR is made not against master, check the PR title.
If the PR title does not match the backport pattern,
post a failure status.

Two ways to resolve:
- reopen the PR against master
- Update the PR title accordingly

Depends on python/devguide#392

Closes python/core-workflow#211
Closes #111
AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
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.

3 participants