Add new rule: vue/define-macros-order (#1855)

[edikdeisling]

No description provided.

[FloEdelmann]

Thank you for this suggestion and implementation. I've only done a preliminary review for now.

[edikdeisling]

There are a few thoughts:

1. Maybe it would be better to have defineEmitsFirst: true rather than order: ['defineEmits', 'defineProps']? Or there are maybe some other define* that could come in?
2. Do we need to give an ability to specify a new line between these statements? newLineBetween: true

[FloEdelmann]

Maybe it would be better to have defineEmitsFirst: true rather than order: ['defineEmits', 'defineProps']? Or there are maybe some other define* that could come in?

I like order better, as it mimics the option from the existing vue/order-in-components rule.

Do we need to give an ability to specify a new line between these statements?

I don't think that should be part of this rule.

[FloEdelmann]

Looks good from my side! :)

@ota-meshi when reviewing, please have a look at these conversations:

• Add new rule: vue/define-macros-order (#1855) #1856 (comment)
• Add new rule: vue/define-macros-order (#1855) #1856 (comment)
• Add new rule: vue/define-macros-order (#1855) #1856 (comment) and Add new rule: vue/define-macros-order (#1855) #1856 (comment)

[ota-meshi]

Thank you for this PR! I wrote some comments.

[ota-meshi]

LGTM! Thank you!

[edikdeisling]

@ota-meshi I found a bug :(
This plugin breaks code with <script> and <script setup> tags in one file.

I'll create PR soon