10000 Fix TypeScript ESLint warning by kunalkundaje · Pull Request #1489 · vuejs/vue-cli · GitHub
[go: up one dir, main page]
Skip to content

Fix TypeScript ESLint warning #1489

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
wants to merge 1 commit into from
Closed

Fix TypeScript ESLint warning #1489

wants to merge 1 commit into from

Conversation

kunalkundaje
Copy link

When generating a TypeScript project with linting enabled, the current set of dependencies resolves to typescript: 2.9.1 and typescript-eslint-parser: 15.0.1, which are not fully compatible with each other and lead to the following warning message when linting:

=============

WARNING: You are currently running a version of TypeScript which is not officially supported by typescript-eslint-parser.

You may find that it works just fine, or you may not.

SUPPORTED TYPESCRIPT VERSIONS: ~2.8.1

YOUR TYPESCRIPT VERSION: 2.9.1

Please only submit bug reports when using the officially supported version.

=============

This PR upgrades typescript-eslint-parser to ^16.0.0 and explicitly upgrades typescript to ^2.9.1 to fix the warning.

@kunalkundaje kunalkundaje changed the title chore(typescript): upgrade dependencies. Fix Typescript ESLint warning Jun 6, 2018
@kunalkundaje kunalkundaje changed the title Fix Typescript ESLint warning Fix TypeScript ESLint warning Jun 6, 2018
@kunalkundaje
Copy link
Author

Looks like this was updated in 7b90fdc
Closing this PR.

@snyk-bot snyk-bot mentioned this pull request Aug 1, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kunalkundaje
0