-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Remove linter option TSLint #5065
Remove linter option TSLint #5065
Conversation
Don't know what to do with the docs: |
I don't know if removing the two tests is the right way, but in the past it seemed that they only passed if |
Thanks for the PR! |
Thanks for your review This PR should aim a cli version 4.2.0. |
Ok ... since some tests use the generator to test linting with TSLint, I will completely revert the second commit |
And because I just |
I just started working on the next major version. |
1bea82e
to
4405277
Compare
@sodatea Thanks for notifying me, I have updated this PR |
The one test Or remove the expectations? |
Yeah, I think we can just remove the two test cases |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I'm happy to be part of the Vue ecosystem and community I am happy to return your thanks to the community ❤️ |
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
Other information:
References #5064