The Wayback Machine - https://web.archive.org/web/20201016233816/https://github.com/vuejs/eslint-plugin-vue/pull/758
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

⭐️New: Add `vue/no-restricted-syntax` rule #758

Merged
merged 3 commits into from Jan 9, 2019

Conversation

@ota-meshi
Copy link
Member

@ota-meshi ota-meshi commented Jan 8, 2019

Close #689

@ota-meshi ota-meshi self-assigned this Jan 8, 2019
Please see [no-restricted-syntax] for detailed options.

You can include the AST created by [vue-eslint-parser] in the selector.
To know more about certain nodes in produced AST, please go [vue-eslint-parser - AST docs].

This comment has been minimized.

@michalsnik

michalsnik Jan 9, 2019
Member

Can you put link to the AST docs here?

This comment has been minimized.

@ota-meshi

ota-meshi Jan 9, 2019
Author Member

It is defined in line 50.


### `"VElement > VExpressionContainer CallExpression"`

Forbind call expressions on mustache interpolation.

This comment has been minimized.

@michalsnik

michalsnik Jan 9, 2019
Member

Forbids call expressions inside mustache interpolation.

]
},

// Forbind call expressions on mustache interpolation

This comment has been minimized.

@michalsnik

michalsnik Jan 9, 2019
Member

Same as above ☝️

@ota-meshi
Copy link
Member Author

@ota-meshi ota-meshi commented Jan 9, 2019

@michalsnik Thank you for your review.
I fixed it. so please check again.

Copy link
Member

@michalsnik michalsnik left a comment

LGTM

@michalsnik michalsnik merged commit 9c49dcc into master Jan 9, 2019
9 checks passed
9 checks passed
Header rules No header rules processed
Details
Pages changed 79 new files uploaded
Details
Redirect rules No redirect rules processed
Details
Mixed content No mixed content detected
Details
ci/circleci: node-v10 Your tests passed on CircleCI!
Details
ci/circleci: node-v10-minimal Your tests passed on CircleCI!
Details
ci/circleci: node-v6 Your tests passed on CircleCI!
Details
ci/circleci: node-v8 Your tests passed on CircleCI!
Details
deploy/netlify Deploy preview ready!
Details
@michalsnik michalsnik deleted the add-rule/no-restricted-syntax branch Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.