Skip to content

Lint for preferring integer literals where double values are expected #34824

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.

Sign up for GitHub

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

Jump to bottom
Closed
mit-mit opened this issue Oct 16, 2018 · 3 comments
Closed

Lint for preferring integer literals where double values are expected #34824

mit-mit opened this issue Oct 16, 2018 · 3 comments
Assignees
@danrubel
Labels
devexp-linter Issues with the analyzer's support for the linter package legacy-area-analyzer Use area-devexp instead. P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug
Milestone
Dart 2.1.1

Comments

@mit-mit
Copy link
Member

mit-mit commented Oct 16, 2018

Dart 2.1 supports a new language features for using integer literals where double values are expected.

It would be great to have a lint for enforcing this in code bases where integer literals are preferred when possible (e.g., something like prefer_integer_interals_as_double_values).
@mit-mit
Copy link
Member Author

mit-mit commented Oct 16, 2018

cc @a14n are you interested in looking at this?

@bwilkerson bwilkerson added legacy-area-analyzer Use area-devexp instead. P3 A lower priority bug or feature request devexp-linter Issues with the analyzer's support for the linter package type-enhancement A request for a change that isn't a bug labels Oct 16, 2018
@a14n
Copy link
Contributor

a14n commented Oct 22, 2018

Fixed by @danrubel via dart-archive/linter#1221

@mit-mit
Copy link
Member Author

mit-mit commented Oct 23, 2018

Excellent, thanks @danrubel

@mit-mit mit-mit added this to the Dart2.1 milestone Oct 23, 2018
@mit-mit mit-mit closed this as completed Oct 23, 2018
@danrubel danrubel mentioned this issue Oct 25, 2018
Closed
@mateusfccp mateusfccp mentioned this issue Nov 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danrubel danrubel

Labels
devexp-linter Issues with the analyzer's support for the linter package legacy-area-analyzer Use area-devexp instead. P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug
Projects
None yet
Milestone
Dart 2.1.1
Development

No branches or pull requests
4 participants
@a14n @danrubel @bwilkerson @mit-mit