Skip to content

Fix for on typo #4318

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

Merged
merged 1 commit into from
Aug 1, 2019
Merged

Fix for on typo #4318

merged 1 commit into from
Aug 1, 2019

Conversation

izik1
Copy link
Contributor

@izik1 izik1 commented Aug 1, 2019

closes #4317

changelog: minor typo fix

@phansch
Copy link
Member

phansch commented Aug 1, 2019

@bors r+ thanks!

@bors
Copy link
Contributor

bors commented Aug 1, 2019

📌 Commit b8a50ca has been approved by phansch

@bors
Copy link
Contributor

bors commented Aug 1, 2019

⌛ Testing commit b8a50ca with merge 9dbab4a...

bors added a commit that referenced this pull request Aug 1, 2019
Fix `for on` typo

<!--
Thank you for making Clippy better!

We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.

If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.

If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.

- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo fmt`

Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.

Delete this line and everything above before opening your PR -->

closes #4317

changelog: minor typo fix
@bors
Copy link
Contributor

bors commented Aug 1, 2019

💔 Test failed - checks-travis

@phansch
Copy link
Member

phansch commented Aug 1, 2019

No worries about the build failure. It's unrelated and being worked on in #4320

@phansch
Copy link
Member

phansch commented Aug 1, 2019

@bors retry

@bors
Copy link
Contributor

bors commented Aug 1, 2019

⌛ Testing commit b8a50ca with merge c8e5352...

bors added a commit that referenced this pull request Aug 1, 2019
Fix `for on` typo

<!--
Thank you for making Clippy better!

We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.

If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.

If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.

- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo fmt`

Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.

Delete this line and everything above before opening your PR -->

closes #4317

changelog: minor typo fix
@bors
Copy link
Contributor

bors commented Aug 1, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing c8e5352 to master...

@bors bors merged commit b8a50ca into rust-lang:master Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Probably Typo in Pedantic (cast_possible_truncation) Lint
3 participants