Skip to content

bpo-29756: Improve documentation for list methods that compare items by equality#572

Merged
zhangyangyu merged 2 commits intopython:masterfrom
zhangyangyu:bpo-29756
Mar 13, 2017
Merged

bpo-29756: Improve documentation for list methods that compare items by equality#572
zhangyangyu merged 2 commits intopython:masterfrom
zhangyangyu:bpo-29756

Conversation

@zhangyangyu
Copy link
Copy Markdown
Member

No description provided.

@zhangyangyu zhangyangyu added docs Documentation in the Doc dir type-feature A feature request or enhancement labels Mar 9, 2017
@mention-bot
Copy link
Copy Markdown

@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.

: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
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.

equals to → equals [or write “is equal to” like above]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks!

: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
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.

Maybe say, Raises a ValueError if there is no such item.

Copy link
Copy Markdown
Member Author

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. :-(

@zhangyangyu zhangyangyu merged commit b2d7717 into python:master Mar 13, 2017
@zhangyangyu zhangyangyu deleted the bpo-29756 branch March 13, 2017 02:09
jaraco pushed a commit that referenced this pull request Dec 2, 2022
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants