Skip to content
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.

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

Removing no-unused-variable for tslint 4.0 #12

Closed
wants to merge 7 commits into from

Conversation

unional
Copy link

@unional unional commented Nov 19, 2016

@blakeembrey blakeembrey mentioned this pull request Nov 20, 2016
@mdurling
Copy link
Contributor

The following rules all have been deprecated and must be removed:

no-unused-variable
no-duplicate-key
no-unreachable

@unional
Copy link
Author

unional commented Nov 21, 2016

Fixed tests

@unional
Copy link
Author

unional commented Nov 21, 2016

vscode-tslint fix is release. Good to go. But seems like tests is not triggered.

Project config issue.

@mdurling
Copy link
Contributor

The tslint-eslint-rules package also needs to be updated and the two additional rules removed. See PR #14

@unional
Copy link
Author

unional commented Nov 21, 2016

Sure. Let's use your PR. Closing this one.

@unional unional closed this Nov 21, 2016
@mdurling
Copy link
Contributor

Travis is failing my CI tests. They pass on my local machine. I had to run npm test:gen to create the .out files. Can you take a look? Am I missing something?

@unional
Copy link
Author

unional commented Nov 21, 2016

Clear your local node_modules and try again. I think the eslint-rules has changed its distribution 👎

@unional
Copy link
Author

unional commented Nov 21, 2016

Oh, nope. It's eslint-rules need to update to 4.0 first.
It is doing require('tslint/lib/lint') which does not exists in 4.0.

@unional
Copy link
Author

unional commented Nov 21, 2016

Ref: buzinas/tslint-eslint-rules#137

@mdurling
Copy link
Contributor

I was getting this error locally until I updated tslint-eslint-rules (which
I included in my PR) and then it worked.

On Mon, Nov 21, 2016 at 11:50 AM Homa Wong notifications@github.com wrote:

Ref: buzinas/tslint-eslint-rules#137
buzinas/tslint-eslint-rules#137


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#12 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADR8_WMYPNOulKEuTVvK1T7OvDSV257Uks5rAcvNgaJpZM4K3Xbb
.

@unional
Copy link
Author

unional commented Nov 21, 2016

I just tried it and 2.2.1 is not working yet.

@unional unional deleted the patch-1 branch November 21, 2016 16:54
@mdurling
Copy link
Contributor

It works on my local machine, but tslint-eslint-rules has its own nested copy of tslint (3.15.1)

screen shot 2016-11-21 at 11 55 26 am

@unional
Copy link
Author

unional commented Nov 21, 2016

It does say unmet deps for me and using 4.0.1, thus the error. Which version of npm are you using?
I'm using 3.10.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants