Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[array-type] autofix is broken #172

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
mysticatea opened this issue Jan 31, 2019 · 0 comments · Fixed by #173
Closed

[array-type] autofix is broken #172

mysticatea opened this issue Jan 31, 2019 · 0 comments · Fixed by #173
Assignees
@armano2
Labels
bug Something isn't working package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin

Comments

@mysticatea
Copy link
Member

Repro

{
  "rules": {
    "@typescript-eslint/array-type": "error"
  }
}
type Unwrap<T> =
    T extends Array<infer E> ? E :
    T extends ReadonlyArray<infer E> ? E :
    /* otherwise */ T

Expected Result

type Unwrap<T> =
    T extends (infer E)[] ? E :
    T extends ReadonlyArray<infer E> ? E :
    /* otherwise */ T

Actual Result

type Unwrap<T> =
    T extends infer E[] ? E :
    T extends ReadonlyArray<infer E> ? E :
    /* otherwise */ T

Additional Info

Nothing in particular.

Versions

package version
@typescript-eslint/eslint-plugin 1.1.1
@typescript-eslint/parser 1.1.1
TypeScript 3.2.2
ESLint 5.12.1
node 11.9.0
npm 6.6.0
@mysticatea mysticatea added package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin triage Waiting for team members to take a look labels Jan 31, 2019
@bradzacher bradzacher added bug Something isn't working and removed triage Waiting for team members to take a look labels Jan 31, 2019
@armano2 armano2 self-assigned this Jan 31, 2019
@armano2 armano2 mentioned this issue Jan 31, 2019
Merged
kaicataldo pushed a commit to kaicataldo/typescript-eslint that referenced this issue Aug 27, 2019
@bradzacher @JamesHenry
[DOCS][no-unused-vars] improve documentation (typescript-eslint#172)
94a1b01
@typescript-eslint typescript-eslint locked as resolved and limited conversation to collaborators Feb 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@armano2 armano2

Labels
bug Something isn't working package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(eslint-plugin): use bracket for infer in array-type rule
3 participants
@armano2 @mysticatea @bradzacher