Movatterモバイル変換

Skip to content

vuejs/eslint-config-typescriptPublic

NotificationsYou must be signed in to change notification settings
Fork37
Star158

Add recommended-type-checked#68

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open

segevfiner wants to merge1 commit intovuejs:main

base:main

Choose a base branch

fromsegevfiner:recommended-type-checked

Open

Add recommended-type-checked#68

segevfiner wants to merge1 commit intovuejs:mainfromsegevfiner:recommended-type-checked

Conversation

segevfiner

Copy link

segevfiner commentedJan 29, 2024•
edited
Loading

This can be used like this:

//@ts-check/* eslint-env node */require('@rushstack/eslint-patch/modern-module-resolution');/**@type {import('eslint').Linter.Config} */module.exports={root:true,extends:['plugin:vue/vue3-recommended','eslint:recommended','@vue/eslint-config-typescript/recommended-type-checked','@vue/eslint-config-prettier',],parserOptions:{ecmaVersion:'latest',tsconfigRootDir:__dirname,project:['./tsconfig.app.json','./tsconfig.vitest.json','./tsconfig.node.json','./e2e/tsconfig.json',],},};

And addingtypescript-eslint-parser-for-extra-files todevDependencies

Seevuejs/vue-eslint-parser#104 &#29

@segevfiner

Add recommended-type-checked

d6c79a2

@segevfiner

Copy link

Author

segevfiner commentedApr 15, 2024

ethanchristensen01

ethanchristensen01 reviewed

View reviewed changes

recommended-type-checked.js

		parser: require.resolve('vue-eslint-parser'),

		parserOptions: {
		parser: require('typescript-eslint-parser-for-extra-files')

Copy link

ethanchristensen01May 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@segevfiner Curiously, what problems were you running into that required this dependency?

Copy link

Author

segevfinerMay 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

The rules that use type checking see the linked issues in the PR description.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Labels

None yet

2 participants

@segevfiner

@ethanchristensen01

[8]ページ先頭

©2009-2025 Movatter.jp