-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
Improve grammar in Glossary. #10474
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
Improve grammar in Glossary. #10474
Conversation
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.
Maybe some of proposed changes are correct, but most of them look incorrect to me. Although I can be wrong.
Doc/whatsnew/3.7.rst
Outdated
@@ -2378,7 +2378,7 @@ Changes in the Python API | |||
instances. | |||
(Contributed by Yury Selivanov in :issue:`32327`.) | |||
|
|||
* :attr:`asyncio.Server.sockets` now returns a copy of the internal list | |||
* :attr:`asyncio.Server.sockets` now return a copy of the internal list |
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.
I don't think this is correct.
Doc/whatsnew/3.7.rst
Outdated
@@ -1223,7 +1223,7 @@ can be set within the scope of a group. | |||
(Contributed by Serhiy Storchaka in :issue:`25054`.) | |||
|
|||
Regular expressions compiled with the :const:`re.LOCALE` flag no longer | |||
depend on the locale at compile time. Locale settings are applied only | |||
depends on the locale at compile time. Locale settings are applied only |
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.
I don't think this is correct.
Doc/whatsnew/3.7.rst
Outdated
@@ -1264,7 +1264,7 @@ The :mod:`socket` module now exposes the :data:`socket.TCP_CONGESTION` | |||
(Contributed by Omar Sandoval in :issue:`26273` and | |||
Nathaniel J. Smith in :issue:`29728`.) | |||
|
|||
Support for :data:`socket.AF_VSOCK` sockets has been added to allow | |||
Support for :data:`socket.AF_VSOCK` sockets have been added to allow |
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.
I have doubts about this.
Doc/whatsnew/3.7.rst
Outdated
@@ -1378,7 +1378,7 @@ This is equivalent to passing :data:`subprocess.PIPE` as *stdout* and | |||
(Contributed by Bo Bayles in :issue:`32102`.) | |||
|
|||
The ``subprocess.run`` function and the :class:`subprocess.Popen` constructor | |||
now accept the *text* keyword argument as an alias | |||
now accepts the *text* keyword argument as an alias |
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.
I have doubts about this.
Doc/whatsnew/3.7.rst
Outdated
@@ -1676,7 +1676,7 @@ Added functions :c:func:`PySlice_Unpack` and :c:func:`PySlice_AdjustIndices`. | |||
:issue:`16500`.) | |||
|
|||
The ``PyExc_RecursionErrorInst`` singleton that was part of the public API | |||
has been removed as its members being never cleared may cause a segfault | |||
has been removed as its members were never cleared may cause a segfault |
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.
I have doubts about this.
Doc/whatsnew/3.7.rst
Outdated
@@ -1855,7 +1855,7 @@ Significant speed improvements to alternate constructors for | |||
constructors when not constructing subclasses. (Contributed by Paul Ganssle | |||
in :issue:`32403`) | |||
|
|||
The speed of comparison of :class:`array.array` instances has been | |||
The speed of comparison of :class:`array.array` instances have been |
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.
I don't think this is correct.
Doc/whatsnew/3.7.rst
Outdated
@@ -1930,7 +1930,7 @@ asyncio | |||
------- | |||
|
|||
Support for directly ``await``-ing instances of :class:`asyncio.Lock` and | |||
other asyncio synchronization primitives has been deprecated. An | |||
other asyncio synchronization primitives have been deprecated. An |
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.
I have doubts about this.
Thank you @serhiy-storchaka, I deleted the one I also not sure. |
I look through the first commit, and I agree with @serhiy-storchaka. The current changeset looks good. |
Thanks @Windsooon for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7. |
a asynchronous generator -> an asynchronous generator (cherry picked from commit a9655b7) Co-authored-by: Windson yang <wiwindson@outlook.com>
GH-10490 is a backport of this pull request to the 3.7 branch. |
GH-10491 is a backport of this pull request to the 3.6 branch. |
a asynchronous generator -> an asynchronous generator (cherry picked from commit a9655b7) Co-authored-by: Windson yang <wiwindson@outlook.com>
a asynchronous generator -> an asynchronous generator (cherry picked from commit a9655b7) Co-authored-by: Windson yang <wiwindson@outlook.com>
a asynchronous generator -> an asynchronous generator (cherry picked from commit a9655b7) Co-authored-by: Windson yang <wiwindson@outlook.com>
a asynchronous generator -> an asynchronous generator