-
-
Notifications
You must be signed in to change notification settings - Fork 669
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
vue/script-indent doesn't respect nested spacing #987
Comments
Thank you for this issue. The I think it works well if you use the @typescript-eslint/indent rule of |
This is so complicated... is there a guide or readme? How are people suppose to find that out? Because |
The |
@ota-meshi vue 3.0 is gonna be released soon and it's based on typescript. While it still has support for JS this would probably inspire peple using typescript for vue. For this issue this means:
|
I know peoples want TypeScript support. |
I added TypeScript AST nodes to KNOWN_NODES constant in the file https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/utils/indent-common.js and it seems to work ok. List of TS AST nodes I added to the end of KNOW_NODES array:
|
@jwayne2 can you publish this? |
Tell us about your environment
Please show your full configuration:
What did you do?
What did you expect to happen?
Since we ban
indent
rule there's no way of check nested spacing. I would expectvue/script-idnent
to warn about incorrect spacing in the same way how wouldindent
rule warn me if it was.ts
fileWhat actually happened?
No warnings are present
Reproduce:
The text was updated successfully, but these errors were encountered: