Skip to content

Commit 552ea4f

Browse files
authored
Allow "default" key to be used also as a Literal (#463)
1 parent a95dfbb commit 552ea4f

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

Diff for: lib/rules/require-default-prop.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ module.exports = {
5050
*/
5151
function propHasDefault (prop) {
5252
const propDefaultNode = prop.value.properties
53-
.find(p => p.key && p.key.name === 'default')
53+
.find(p =>
54+
p.key &&
55+
(p.key.name === 'default' || p.key.value === 'default')
56+
)
5457

5558
return Boolean(propDefaultNode)
5659
}

Diff for: tests/lib/rules/require-default-prop.js

+8-3
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,16 @@ ruleTester.run('require-default-prop', rule, {
3939
},
4040
c: {
4141
type: Number,
42-
default: 0,
43-
required: false
42+
required: false,
43+
default: 0
44+
},
45+
d: {
46+
type: String,
47+
required: false,
48+
'default': 'lorem'
4449
},
4550
// eslint-disable-next-line require-default-prop
46-
d: Number
51+
e: Number
4752
}
4853
}
4954
`,

0 commit comments

Comments
 (0)