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
bpo-29756: Improve documentation for list methods that compare items by equality #572
Conversation
@zhangyangyu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @ncoghlan, @benjaminp, @ezio-melotti and @tiran to be potential reviewers. |
Doc/tutorial/datastructures.rst
Outdated
@@ -40,7 +40,7 @@ objects: | |||
.. method:: list.remove(x) | |||
:noindex: | |||
|
|||
Remove the first item from the list whose value is *x*. It is an error if | |||
Remove the first item from the list whose value equals to *x*. It is an error if |
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.
equals to → equals [or write “is equal to” like above]
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.
Thanks!
Doc/tutorial/datastructures.rst
Outdated
@@ -40,7 +40,7 @@ objects: | |||
.. method:: list.remove(x) | |||
:noindex: | |||
|
|||
Remove the first item from the list whose value is *x*. It is an error if | |||
Remove the first item from the list whose value equals to *x*. It is an error if |
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 say, Raises a ValueError if there is no such item
.
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.
In my mind they don't have any difference. :-(
Bumps [sentry-sdk](https://github.com/getsentry/sentry-python) from 1.8.0 to 1.9.0. - [Release notes](https://github.com/getsentry/sentry-python/releases) - [Changelog](https://github.com/getsentry/sentry-python/blob/master/CHANGELOG.md) - [Commits](getsentry/sentry-python@1.8.0...1.9.0) --- updated-dependencies: - dependency-name: sentry-sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
No description provided.