Skip to content

Commit 6634c2d

Browse files
committed
feature: do not handle array prop declaration (vuejs#2465)
1 parent da17d74 commit 6634c2d

File tree

2 files changed

+1
-15
lines changed

2 files changed

+1
-15
lines changed

Diff for: lib/rules/define-props-declaration.js

+1-10
Original file line numberDiff line numberDiff line change
@@ -99,16 +99,7 @@ const mapNativeType = (nativeType) => {
9999
*/
100100
function getComponentPropData(prop, sourceCode) {
101101
if (prop.type === 'array') {
102-
if (prop.node.type !== 'Identifier') {
103-
throw new Error(`Unexpected prop type inside array: ${prop.node.type}`)
104-
}
105-
106-
return {
107-
name: prop.node.name,
108-
type: 'string',
109-
required: false,
110-
defaultValue: undefined
111-
}
102+
throw new Error(`Unable to resolve types based on array prop declaration.`)
112103
} else if (prop.type === 'object') {
113104
if (prop.propName === null) {
114105
throw new Error('Unexpected prop with null name.')

Diff for: tests/lib/rules/define-props-declaration.js

-5
Original file line numberDiff line numberDiff line change
@@ -642,11 +642,6 @@ tester.run('define-props-declaration', rule, {
642642
const props = defineProps([kind])
643643
</script>
644644
`,
645-
output: `
646-
<script setup lang="ts">
647-
const props = defineProps<{ kind?: string }>()
648-
</script>
649-
`,
650645
errors: [
651646
{
652647
message: 'Use type-based declaration instead of runtime declaration.',

0 commit comments

Comments
 (0)