Skip to content

Commit 45be306

Browse files
jastkandmysticatea
authored andcommitted
Fix: no-unused-components rule should allow camelCase (#606)
1 parent dd6ed4f commit 45be306

File tree

2 files changed

+18
-3
lines changed

2 files changed

+18
-3
lines changed

Diff for: lib/rules/no-unused-components.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -72,13 +72,13 @@ module.exports = {
7272

7373
registeredComponents
7474
.filter(({ name }) => {
75-
// If the component name is PascalCase
75+
// If the component name is PascalCase or camelCase
7676
// it can be used in various of ways inside template,
7777
// like "theComponent", "The-component" etc.
7878
// but except snake_case
79-
if (casing.pascalCase(name) === name) {
79+
if (casing.pascalCase(name) === name || casing.camelCase(name) === name) {
8080
return !usedComponents.some(n => {
81-
return n.indexOf('_') === -1 && name === casing.pascalCase(n)
81+
return n.indexOf('_') === -1 && (name === casing.pascalCase(n) || casing.camelCase(n) === name)
8282
})
8383
} else {
8484
// In any other case the used component name must exactly match

Diff for: tests/lib/rules/no-unused-components.js

+15
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,21 @@ tester.run('no-unused-components', rule, {
127127
}
128128
</script>`
129129
},
130+
{
131+
filename: 'test.vue',
132+
code: `<template>
133+
<div>
134+
<the-button />
135+
</div>
136+
</template>
137+
<script>
138+
export default {
139+
components: {
140+
theButton
141+
}
142+
}
143+
</script>`
144+
},
130145
{
131146
filename: 'test.vue',
132147
code: `<template>

0 commit comments

Comments
 (0)