Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update backport instructions#178

Merged
willingc merged 3 commits into
masterpython/devguide:masterfrom
issue-173python/devguide:issue-173Copy head branch name to clipboard
Apr 25, 2017
Merged

Update backport instructions#178
willingc merged 3 commits into
masterpython/devguide:masterfrom
issue-173python/devguide:issue-173Copy head branch name to clipboard

Conversation

@Mariatta

Copy link
Copy Markdown
Member

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

@willingc willingc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great @Mariatta. One small question.

Comment thread committing.rst Outdated

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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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

@Mariatta

Copy link
Copy Markdown
Member Author

Thanks :) Updated.

@willingc willingc merged commit 8f44690 into master Apr 25, 2017
@Mariatta Mariatta deleted the issue-173 branch April 25, 2017 17:33
@berkerpeksag

Copy link
Copy Markdown
Member

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

@willingc

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Member Author

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

AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
* Update backport instructions

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 python#173

* hash -> hashes

* add `an`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update the section about backporting changes

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.