Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
c7de522
First draft: make text easier to skim
jablonskidev Dec 23, 2020
b6acaca
Change list spacing: learning rst
jablonskidev Dec 23, 2020
a3a92f7
Learning rst
jablonskidev Dec 23, 2020
e0742b1
Learning rst
jablonskidev Dec 23, 2020
0bd9226
Fix list indentation
jablonskidev Dec 23, 2020
6e26e24
Remove monospace when conflicted with bold
jablonskidev Dec 23, 2020
5be3269
Allcaps for NEWS
jablonskidev Dec 23, 2020
ff18318
Sentence case for subheadings
jablonskidev Dec 23, 2020
1f7687c
Allcaps NEWS
jablonskidev Dec 23, 2020
2b91e10
Remove inconsistent bold
jablonskidev Dec 23, 2020
ef292e0
Reader-focused phrasing
jablonskidev Dec 23, 2020
e6b848b
Verb tense
jablonskidev Dec 23, 2020
f4e9e83
Add empty lines between list items: learn rst
jablonskidev Dec 24, 2020
39d40d8
Add whitespace: learn rst
jablonskidev Dec 24, 2020
ed2dc39
Remove double empty lines between list items
jablonskidev Dec 24, 2020
4634697
Replace "PR" with "pull request"
jablonskidev Dec 24, 2020
64de2c8
Line length and spacing
jablonskidev Dec 24, 2020
cfae673
Typo & consistent terminology
jablonskidev Dec 24, 2020
9221842
Light proofread
jablonskidev Dec 24, 2020
25a885a
Made line under heading long enough
jablonskidev Dec 24, 2020
31ed26d
Add link to CLA
jablonskidev Dec 24, 2020
e817721
Changed ordered list to unordered list
jablonskidev Dec 24, 2020
1209680
Change list formatting
jablonskidev Dec 24, 2020
a79e2d1
Removed empty lines in list
jablonskidev Dec 24, 2020
d543c51
Changed bolding
jablonskidev Dec 24, 2020
dc087d3
Changed line length for list formatting
jablonskidev Dec 24, 2020
da180f8
Changed word order for clarity without monospace
jablonskidev Dec 24, 2020
567aa81
Added monospace to bold sections
jablonskidev Dec 24, 2020
2271326
Font formatting
jablonskidev Dec 24, 2020
623cea4
Removed redundancy
jablonskidev Dec 24, 2020
10e28fc
Monospace NEWS, clean double empty lines
jablonskidev Dec 24, 2020
9caaf03
Font formatting
jablonskidev Dec 24, 2020
9f7ca66
Spelling & font formatting
jablonskidev Dec 24, 2020
3b530a9
Font formatting
jablonskidev Dec 24, 2020
afb0c74
Added a reference to b.p.o.
jablonskidev Dec 28, 2020
6c7cf90
Fixed monospace
jablonskidev Dec 28, 2020
864c27a
Removed instructions to run patchcheck
jablonskidev Jan 4, 2021
f4e718a
Put patchcheck instructions back in
jablonskidev Jan 5, 2021
76aea19
Corrected wording in the reference to b.p.o.
jablonskidev Jan 9, 2021
d1b5acf
Implemented Brett's feedback
jablonskidev Jan 10, 2021
91f046e
Addressed issue #655
jablonskidev Jan 23, 2021
6c46cfb
Update committing.rst
brettcannon Jan 30, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add empty lines between list items: learn rst
  • Loading branch information
jablonskidev authored Dec 24, 2020
commit f4e9e83ea3cf97607ca04a360f84bfcad78ce327
12 changes: 12 additions & 0 deletions committing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -18,35 +18,43 @@ the public source tree. Ask yourself the following questions:
only branch that receives new features is ``master``, the in-development
branch.


#. **Are there comments on the PR?** Look for explanations about whether
the code coverage increased or stayed the same.


#. **Are the changes acceptable?** If you want to share your work-in-progress
code on a feature or bugfix, then you can open a ``WIP``-prefixed PR,
publish patches on the `issue tracker <https://bugs.python.org/>`_, or
create a public fork of the repository.


#. **Does the check on the PR show that the test suite passes?** :ref:`Run the
entire test suite <runtests>` to ensure that it passes. Running a single
test is not enough since the changes may have unforeseen effects on other
tests or library modules.


#. **Is the patch in a good state?** Check :ref:`patch` and
:ref:`helptriage` to review what is expected of a patch.


#. **Do the changes meet the requirements of the patch checklist?** :ref:`Run
patchcheck <patchcheck>` to perform a quick confidence check on the changes.


#. **Does the patch break backwards-compatibility without a strong
reason?** :ref:`Run the entire test suite <runtests>` to
make sure that everything still passes. If there is a change to the
semantics, then there needs to be a strong reason, because it will
cause some peoples' code to break. If you are unsure if the breakage
is worth it, then ask on python-dev.


#. **Were configure and pyconfig.h.in regenerated?**
Regnenerate them if necessary.


#. **Were appropriate labels added to signify necessary backporting of the
pull request?** If it is determined that a pull request needs to be
backported into one or more of the maintenance branches, then a core
Expand All @@ -56,6 +64,7 @@ the public source tree. Ask yourself the following questions:
core developers and members of the `Python Triage Team`_ can apply
labels to GitHub pull requests).


#. **Does the PR have a label indicating that the submitter has signed the
CLA?** Make sure that the contributor has signed a `Contributor Licensing
Agreement`_ (CLA), unless their change has no possible intellectual
Expand All @@ -65,6 +74,7 @@ the public source tree. Ask yourself the following questions:
put in their GitHub username. For further questions about the CLA
process, write to: contributors@python.org.


#. **Was the contributor added to Misc/ACKS?** Make sure that
the patch is attributed correctly with the contributor's name in
``Misc/ACKS``. If the patch has been heavily modified, then "Initial
Expand All @@ -74,6 +84,7 @@ the public source tree. Ask yourself the following questions:
in a commit. Add ``Co-authored-by: name <name@example.com>`` at the end
of the commit message.


#. **Were What's New in Python (in Doc/whatsnew/) and
Misc/NEWS.d/next updated?** If the change is particularly interesting
for end users (e.g. new features, significant improvements, or
Expand All @@ -82,6 +93,7 @@ the public source tree. Ask yourself the following questions:
as well. Changes that affect only documentation generally do not require
a NEWS entry. (See the following section for more information.)


Updating NEWS and What's New in Python
--------------------------------------

Expand Down