Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Move some unreadable_literal ui tests to correct file #3887

Closed

Conversation

phansch
Copy link
Member

@phansch phansch commented Mar 15, 2019

The moved tests were part of tests/ui/literals.rs before. Now they are
placed in tests/ui/unreadable_literal.rs which now also runs
rustfix on these 4 extra let statements.

cc #3603, #2038

The moved tests were part of `tests/ui/literals.rs` before. Now they are
placed in `tests/ui/unreadable_literal.rs`, which now also runs
`rustfix` on these 4 extra let statements.
phansch added a commit to phansch/rust-clippy that referenced this pull request Mar 15, 2019
This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc rust-lang#3603, rust-lang#2038

Based on rust-lang#3887
@phansch phansch closed this Mar 15, 2019
bors added a commit that referenced this pull request Mar 18, 2019
…nd_rustfix, r=flip1995

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887
phansch added a commit to phansch/rust-clippy that referenced this pull request Mar 19, 2019
This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc rust-lang#3603, rust-lang#2038

Based on rust-lang#3887
bors added a commit that referenced this pull request Mar 19, 2019
…nd_rustfix, r=flip1995

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant