Skip to content

Fix false positive for script setup in vue/no-expose-after-await rule#1896

Merged
ota-meshi merged 1 commit intomasterfrom
issue1895
May 18, 2022
Merged

Fix false positive for script setup in vue/no-expose-after-await rule#1896
ota-meshi merged 1 commit intomasterfrom
issue1895

Conversation

@ota-meshi
Copy link
Copy Markdown
Member

fixes #1895

callNode.parent.type === 'ExpressionStatement' &&
callNode.parent.parent === node,
callNode.parent.parent === node &&
id.name === 'defineExpose',
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I should have added an id check but missed it.

@ota-meshi
Copy link
Copy Markdown
Member Author

I will merge and release this as it has a big impact.

@ota-meshi ota-meshi merged commit 5a5c6e8 into master May 18, 2022
@ota-meshi ota-meshi deleted the issue1895 branch May 18, 2022 13:54
@FloEdelmann
Copy link
Copy Markdown
Member

Retrospective approval 😉

Thanks for fixing!

@AndreasNasman
Copy link
Copy Markdown

Same, thanks! ❤️

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vue/no-expose-after-await false positives

3 participants