Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

no-angle-bracket-type-assertion --fix misses cases that need extra parenthesis #2955

Closed
JoshuaKGoldberg opened this issue Jun 26, 2017 · 0 comments · Fixed by #3301
Closed

no-angle-bracket-type-assertion --fix misses cases that need extra parenthesis #2955

JoshuaKGoldberg opened this issue Jun 26, 2017 · 0 comments · Fixed by #3301
Labels
P2 Status: Accepting PRs Type: Bug

Comments

@JoshuaKGoldberg
Copy link
Contributor

Bug Report

  • TSLint version: 5.4.0
  • TypeScript version: 2.4 RC
  • Running TSLint via: (pick one) CLI

TypeScript code being linted

<number><any>flag & value

with tslint.json configuration:

{
    "rules": {
        "no-angle-bracket-type-assertion": true
    }
}

Actual behavior

flag as any as number & value

Expected behavior

(flag as any as number) & value
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Jun 26, 2017
Closed
@ajafff ajafff mentioned this issue Oct 8, 2017
Merged
4 tasks
adidahiya pushed a commit that referenced this issue Oct 20, 2017
@ajafff @adidahiya
no-angle-bracket-type-assertion: fixed replacement order (#3301)
e599184 
... and add parens when necessary

Fixes: #2957
Fixes: #2955
HyphnKnight pushed a commit to HyphnKnight/tslint that referenced this issue Apr 9, 2018
@ajafff @HyphnKnight
no-angle-bracket-type-assertion: fixed replacement order (palantir#3301)
742497e 
... and add parens when necessary

Fixes: palantir#2957
Fixes: palantir#2955
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P2 Status: Accepting PRs Type: Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

no-angle-bracket-type-assertion: fixed replacement order and add parens
2 participants
@adidahiya @JoshuaKGoldberg