Skip to content
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

Update backport instructions #178

Merged
merged 3 commits into from Apr 25, 2017
Merged

Update backport instructions #178

merged 3 commits into from Apr 25, 2017

Conversation

@Mariatta
Copy link
Member

@Mariatta Mariatta commented Apr 25, 2017

Remove the part about merging into master.
Remove the part about prefixing the PR title with X.Y, cherry_picker.py already does this.

Closes #173

Remove the part about merging into master.
Remove the part about prefixing the PR title with `X.Y`, cherry_picker.py already does this.

Closes #173
Copy link
Collaborator

@willingc willingc left a comment

Looks great @Mariatta. One small question.

@@ -408,45 +408,25 @@ new features. The other branches only receive bug fixes or security fixes.

.. _branch-merge:

Backporting changes to Python 3.6 (or older version)
----------------------------------------------------
Backporting Changes to Older Version

This comment has been minimized.

@willingc

willingc Apr 25, 2017
Collaborator

@Mariatta: Should there be an an before Older Version?

@Mariatta
Copy link
Member Author

@Mariatta Mariatta commented Apr 25, 2017

Thanks :) Updated.

@willingc willingc merged commit 8f44690 into master Apr 25, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Mariatta Mariatta deleted the issue-173 branch Apr 25, 2017
@berkerpeksag
Copy link
Member

@berkerpeksag berkerpeksag commented Apr 25, 2017

I think the part about prefixing PR title should stay for now. Not every core dev use cherry_picker.py yet :)

@willingc
Copy link
Collaborator

@willingc willingc commented Apr 25, 2017

A reasonable comment during the transition @berkerpeksag. Perhaps adding a small note about prefixing the PR title and marking it "legacy process" and the cherry_picker.py as the "preferred process".

@Mariatta thoughts?

@Mariatta
Copy link
Member Author

@Mariatta Mariatta commented Apr 25, 2017

Sounds reasonable :) I'm reviewing the other PR now, and will add back the PR prefix after.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.