Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

Implement unnecessary-type-assertion #40

Closed
Anrock opened this issue Jun 11, 2020 · 2 comments
Closed

Implement unnecessary-type-assertion #40

Anrock opened this issue Jun 11, 2020 · 2 comments
Assignees
@dkrutskikh
Labels
duplicate This issue or pull request already exists type: enhancement New feature or request

Comments

@Anrock
Copy link
Contributor

Anrock commented Jun 11, 2020

Issue a warning for unnecessary type assertions.

Bad:

final myList = <int>[1,2,3];
...
myList.whereType<int> ... // myList is known to contain ints, therefore type assertion is redundant
...
if (myList is List) ... // myList is known to be a List, therefore type check is redundant

Inspired by TSLint rule
@dkrutskikh dkrutskikh added the type: enhancement New feature or request label Jun 23, 2020
@dkrutskikh dkrutskikh self-assigned this Oct 24, 2020
@roman-petrov
Copy link
Contributor

Just linking similar issue dart-lang/linter#2591

@dkrutskikh dkrutskikh added the waiting for release Will be available after new version is released label Nov 18, 2021
@incendial
Copy link
Member

Implemented here: #477

@incendial incendial added duplicate This issue or pull request already exists and removed waiting for release Will be available after new version is released labels Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dkrutskikh dkrutskikh

Labels
duplicate This issue or pull request already exists type: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Anrock @dkrutskikh @roman-petrov @incendial