vuejs/eslint-plugin-vuePublic

NotificationsYou must be signed in to change notification settings
Fork680
Star4.5k

feat: add`vue/no-import-compiler-macros` rule#2684

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

Already on GitHub?Sign in to your account

Jump to bottom

Merged

ota-meshi merged 9 commits intomasterfromno-import-compiler-macros

Mar 5, 2025

Merged

feat: add`vue/no-import-compiler-macros` rule#2684

ota-meshi merged 9 commits intomasterfromno-import-compiler-macros

Mar 5, 2025

Conversation

Copy link

Member

waynzh commentedFeb 13, 2025

resolve#2437

FloEdelmann requested changes

Feb 13, 2025

View reviewed changes

Copy link

Member

FloEdelmann left a comment

Choose a reason for hiding this comment

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

Thanks for adding this rule! 🙂

lib/rules/no-import-compiler-macros.js OutdatedShow resolvedHide resolved

docs/rules/no-import-compiler-macros.mdShow resolvedHide resolved

lib/rules/no-import-compiler-macros.js OutdatedShow resolvedHide resolved

waynzhand others added9 commits

February 20, 2025 19:46

feat: add no-import-compiler-macros rule

87b05ba

feat: add docs

d692fdd

Update lib/rules/no-import-compiler-macros.js

8c4c5c6

Co-authored-by: Flo Edelmann <git@flo-edelmann.de>

Update docs

ce38578

refactor

eb0d6d2

refactor: autofix

781692c

feat: add test

f539191

refactor

8b746bb

rename

3c7654d

waynzh force-pushed theno-import-compiler-macros branch from3562234 to3c7654dCompare

February 20, 2025 11:47

waynzh requested a review fromFloEdelmann

February 20, 2025 11:50

FloEdelmann approved these changes

Feb 20, 2025

View reviewed changes

Copy link

Member

FloEdelmann left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks 🙂

FloEdelmann changed the title~~feat: add no-import-compiler-macros rule~~feat: addvue/no-import-compiler-macros rule

Feb 20, 2025

waynzh requested a review fromota-meshi

March 3, 2025 13:09

ota-meshi approved these changes

Mar 5, 2025

View reviewed changes

Copy link

Member

ota-meshi left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

ota-meshi merged commitddfd676 intomaster

Mar 5, 2025

12 checks passed

ota-meshi deleted the no-import-compiler-macros branch

March 5, 2025 04:20

Labels

None yet

3 participants

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

feat: add`vue/no-import-compiler-macros` rule#2684

feat: add`vue/no-import-compiler-macros` rule#2684

Uh oh!

Conversation

waynzh commentedFeb 13, 2025

Uh oh!

FloEdelmann left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

FloEdelmann left a comment

Choose a reason for hiding this comment

Uh oh!

ota-meshi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Movatterモバイル変換

Uh oh!

feat: addvue/no-import-compiler-macros rule#2684

feat: addvue/no-import-compiler-macros rule#2684

Uh oh!

Conversation

waynzh commentedFeb 13, 2025

Uh oh!

FloEdelmann left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

FloEdelmann left a comment

Choose a reason for hiding this comment

Uh oh!

ota-meshi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

feat: add`vue/no-import-compiler-macros` rule#2684

feat: add`vue/no-import-compiler-macros` rule#2684