-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
bpo-32523: Simplifying news entries with multiple paragraphs. #8154
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -196,7 +196,6 @@ ImportError rather than SystemError. | |
.. section: Core and Builtins | ||
Improve signal delivery. | ||
|
||
Avoid using Py_AddPendingCall from signal handler, to avoid calling | ||
signal-unsafe functions. The tests I'm adding here fail without the rest of | ||
the patch, on Linux and OS X. This means our signal delivery logic had | ||
|
@@ -596,7 +595,6 @@ Fix out of bounds write in `asyncio.CFuture.remove_done_callback()`. | |
.. section: Library | ||
signal.setitimer() may disable the timer when passed a tiny value. | ||
|
||
Tiny values (such as 1e-6) are valid non-zero values for setitimer(), which | ||
is specified as taking microsecond-resolution intervals. However, on some | ||
platform, our conversion routine could convert 1e-6 into a zero interval, | ||
|
@@ -774,7 +772,6 @@ Add a missing xmlns to python.manifest so that it matches the schema. | |
.. section: IDLE | ||
IDLE code context -- fix code update and font update timers. | ||
|
||
Canceling timers prevents a warning message when test_idle completes. | ||
|
||
.. | ||
|
@@ -817,7 +814,6 @@ IDLE - make tests pass with zzdummy extension disabled by default. | |
Document how IDLE runs tkinter programs. IDLE calls tcl/tk update in the | ||
background in order to make live | ||
|
||
interaction and experimentation with tkinter applications much easier. | ||
|
||
.. | ||
|
@@ -847,7 +843,6 @@ Rearrange IDLE configdialog GenPage into Window, Editor, and Help sections. | |
.. section: IDLE | ||
IDLE - Add docstrings and tests for outwin subclass of editor. | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. With hg, news items were expected to be the same as commit messages, and commit messages were expected to have a blank line after the title/subject line. This is still recommended for git commit messages, but apparently is handled for us in github by having separate title and message boxes when doing the squash merge commit. I initially continued adding blank lines because I noticed that without them the message was re-wrapped so that there was no separate subject line, contrary to how news entries were once expected to look. Once I learned that NEWS entries were now supposed to be a single paragraph without a separate header, I stopped the blanks. |
||
Move some data and functions from the class to module level. Patch by Cheryl | ||
Sabella. | ||
|
||
|
@@ -868,24 +863,20 @@ IDLE - Do not modify tkinter.message in test_configdialog. | |
.. section: IDLE | ||
Convert IDLE's built-in 'extensions' to regular features. | ||
|
||
About 10 IDLE features were implemented as supposedly optional extensions. | ||
Their different behavior could be confusing or worse for users and not good | ||
for maintenance. Hence the conversion. | ||
|
||
The main difference for users is that user configurable key bindings for | ||
builtin features are now handled uniformly. Now, editing a binding in a | ||
keyset only affects its value in the keyset. All bindings are defined | ||
together in the system-specific default keysets in config-extensions.def. | ||
All custom keysets are saved as a whole in config-extension.cfg. All take | ||
effect as soon as one clicks Apply or Ok. | ||
|
||
The affected events are '<<force-open-completions>>', '<<expand-word>>', | ||
'<<force-open-calltip>>', '<<flash-paren>>', '<<format-paragraph>>', | ||
'<<run-module>>', '<<check-module>>', and '<<zoom-height>>'. Any (global) | ||
customizations made before 3.6.3 will not affect their keyset-specific | ||
customization after 3.6.3. and vice versa. | ||
|
||
Inital patch by Charles Wohlganger. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since Ned complained about a long IDLE news message for 3.7.0b5 or .0rc1, I started putting the long form message I think current users really need to see, like the above, in idlelib/NEWS.txt, viewed on About IDLE, and a condensed form in the general news entry. I also am adding more IDLE entries in What's New, including some x.y.z entries. Putting acknowledgements on a new line, sometimes separated by a blank, was also standard practice. I mostly did not insert a blank except in a case like this. I see that others often did. |
||
|
||
.. | ||
|
@@ -974,7 +965,6 @@ continue to pass. Patch by Cheryl Sabella. | |
.. section: IDLE | ||
IDLE - Factor FontPage(Frame) class from ConfigDialog. | ||
|
||
Slightly modified tests continue to pass. Fix General tests. Patch mostly by | ||
Cheryl Sabella. | ||
|
||
|
@@ -1007,7 +997,6 @@ the tabs and will enable splitting the groups into classes. | |
.. section: IDLE | ||
IDLE -- Factor a VarTrace class out of ConfigDialog. | ||
|
||
Instance tracers manages pairs consisting of a tk variable and a callback | ||
function. When tracing is turned on, setting the variable calls the | ||
function. Test coverage for the new class is 100%. | ||
|
@@ -1029,12 +1018,10 @@ IDLE: Add more tests for General tab. | |
.. section: IDLE | ||
IDLE - Improve configdialog font page and tests. | ||
|
||
In configdialog: Document causal pathways in create_font_tab docstring. | ||
Simplify some attribute names. Move set_samples calls to var_changed_font | ||
(idea from Cheryl Sabella). Move related functions to positions after the | ||
create widgets function. | ||
|
||
In test_configdialog: Fix test_font_set so not order dependent. Fix renamed | ||
test_indent_scale so it tests the widget. Adjust tests for movement of | ||
set_samples call. Add tests for load functions. Put all font tests in one | ||
|
@@ -1077,12 +1064,9 @@ Patch by Louie Lu. | |
.. section: IDLE | ||
Document coverage details for idlelib tests. | ||
|
||
* Add section to idlelib/idle-test/README.txt. | ||
|
||
* Include check that branches are taken both ways. | ||
|
||
* Exclude IDLE-specific code that does not run during unit tests. | ||
Add section to idlelib/idle-test/README.txt. | ||
Include check that branches are taken both ways. | ||
Exclude IDLE-specific code that does not run during unit tests. | ||
|
||
.. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This extra line was an error.