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
Minor grammar fixes #1338
Minor grammar fixes #1338
Conversation
Generally just a few misspelling mistakes. A few re-wording changes but nothing major.
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.
Is this the result of applying a grammar tool?
I'm tempted to reject on the grounds that it's wasting time, given that the changes don't significantly affect readability or content.
`rand_distr` each contain a lot of functionality. | ||
- Simple (implementation). We have a strong focus on correctness, speed and flexibility, but | ||
`rand_distr` each contains a lot of functionality. | ||
- Simple (implementation). We have a strong focus on correctness, speed, and flexibility, but |
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 have anything against use of an Oxford comma here, but it's a matter of style rather than correctness.
@@ -60,7 +60,7 @@ To get started using Rand, see [The Book](https://rust-random.github.io/book). | |||
## Versions | |||
|
|||
Rand is *mature* (suitable for general usage, with infrequent breaking releases | |||
which minimise breakage) but not yet at 1.0. We maintain compatibility with | |||
that minimize breakage) but not yet at 1.0. We maintain compatibility with |
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.
@@ -70,18 +70,18 @@ Current Rand versions are: | |||
changes and fixes. | |||
- Version 0.8 was released in December 2020 with many small changes. | |||
|
|||
A detailed [changelog](CHANGELOG.md) is available for releases. | |||
A detailed [changelog](CHANGELOG.md) is available for release. |
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 isn't a grammar change: it's a change of meaning. Write "for each release" if you prefer.
[semver trick](https://github.com/dtolnay/semver-trick/) in order to make traits | ||
[semver trick](https://github.com/dtolnay/semver-trick/) to make traits |
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.
The result is harder to parse. Revert please.
Generally just a few misspelling mistakes. A few re-wording changes but nothing major.