Skip to content

Update wording on committing.rst. - #262

Merged
ezio-melotti merged 2 commits into
python:masterfrom
ezio-melotti:rename-committing
Sep 6, 2017
Merged

Update wording on committing.rst.#262
ezio-melotti merged 2 commits into
python:masterfrom
ezio-melotti:rename-committing

Conversation

@ezio-melotti

Copy link
Copy Markdown
Member

No description provided.

Comment threadcommitting.rst Outdated

Is the change ready for committing?
-----------------------------------
This page is aimed to core developers, and cover the steps required to

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

cover -> covers

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, covers :)

Comment threadcommitting.rst
@@ -1,18 +1,17 @@
.. _committing:

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Should the filename and the label be updated too?

Comment threadcommitting.rst
Commit Style
------------

.. move this to pullrequest

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Committing is now done by contributors, so this section doesn't belong here anymore.
Something could be said about the commit message used while squashing.

@ezio-melotti
ezio-melotti merged commit 4207d69 into python:masterSep 6, 2017
@ezio-melotti
ezio-melotti deleted the rename-committing branch September 6, 2017 17:27
AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
* Update wording on committing.rst.
* Fix typo.
Sign up for freeto 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.

3 participants

@ezio-melotti@Mariatta@the-knights-who-say-ni