Skip to content
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

Move check for lossy whole-number floats out of excessive_precision #5202

Conversation

krishna-veerareddy
Copy link
Contributor

@krishna-veerareddy krishna-veerareddy commented Feb 20, 2020

changelog: Add new lint lossy_float_literal to detect lossy whole number float literals and move it out of excessive_precision again.

Fixes #5201

tests/ui/lossy_float_literal.rs Show resolved Hide resolved
clippy_lints/src/lossy_float_literal.rs Outdated Show resolved Hide resolved
clippy_lints/src/excessive_precision.rs Show resolved Hide resolved
@krishna-veerareddy krishna-veerareddy force-pushed the issue-5201-move-lossy-float-literal-restriction branch from f2a486b to c25f1ea Compare February 21, 2020 02:39
@krishna-veerareddy krishna-veerareddy force-pushed the issue-5201-move-lossy-float-literal-restriction branch from c25f1ea to 219c94d Compare February 21, 2020 06:33
Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

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

Thanks!

@flip1995
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Feb 21, 2020

📌 Commit 219c94d has been approved by flip1995

@bors
Copy link
Contributor

bors commented Feb 21, 2020

⌛ Testing commit 219c94d with merge f8ea194...

bors added a commit that referenced this pull request Feb 21, 2020
…literal-restriction, r=flip1995

Move check for lossy whole-number floats out of `excessive_precision`

changelog: Add new lint `lossy_float_literal` to detect lossy whole number float literals and move it out of `excessive_precision` again.

Fixes #5201
@bors
Copy link
Contributor

bors commented Feb 21, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Feb 21, 2020

⌛ Testing commit 219c94d with merge 6e82b89...

bors added a commit that referenced this pull request Feb 21, 2020
…literal-restriction, r=flip1995

Move check for lossy whole-number floats out of `excessive_precision`

changelog: Add new lint `lossy_float_literal` to detect lossy whole number float literals and move it out of `excessive_precision` again.

Fixes #5201
@bors
Copy link
Contributor

bors commented Feb 21, 2020

💔 Test failed - checks-action_test

@flip1995
Copy link
Member

@bors retry (MacOS toolchain download...)

@bors
Copy link
Contributor

bors commented Feb 21, 2020

⌛ Testing commit 219c94d with merge acfcbee...

@bors
Copy link
Contributor

bors commented Feb 21, 2020

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing acfcbee to master...

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.

Broken suggestion from excessive_precision lint
3 participants