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

no-any: remove fixer, amend error message #3486

Merged
merged 1 commit into from
Nov 16, 2017
Merged

no-any: remove fixer, amend error message #3486

merged 1 commit into from
Nov 16, 2017

Conversation

ajafff
Copy link
Contributor

@ajafff ajafff commented Nov 14, 2017

PR checklist

Overview of change:

Removes the fixer and amends the error message as suggested in #2491
Fixes: #2491

Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

[bugfix] removed fixer of no-any

Removes the fixer and amends the error message as suggested in #2491
Fixes: #2491

[rule-change] removed fixer of `no-any`
@ChrisMBarr
Copy link
Contributor

Yes thanks you! This makes this rule a lot less susceptible to errors by uninformed developers that would run into this message.

@ajafff ajafff merged commit cbdcfb5 into palantir:master Nov 16, 2017
@ajafff ajafff deleted the no-any-fixer branch November 16, 2017 07:46
HyphnKnight pushed a commit to HyphnKnight/tslint that referenced this pull request Apr 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bad suggestions from the no-any rule
3 participants