Skip to content

Commit 1942f6d

Browse files
committed
Move used prop type flag setting to new helper
1 parent fa0cb1c commit 1942f6d

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

lib/util/propTypes.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -666,13 +666,6 @@ module.exports = function propTypesInstructions(context, components, utils) {
666666
}
667667
},
668668

669-
JSXSpreadAttribute: function(node) {
670-
const component = components.get(utils.getParentComponent());
671-
components.set(component ? component.node : node, {
672-
ignoreUnusedPropTypesValidation: true
673-
});
674-
},
675-
676669
TypeAlias: function(node) {
677670
setInTypeScope(node.id.name, node.right);
678671
},

lib/util/usedPropTypes.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -488,6 +488,13 @@ module.exports = function usedPropTypesInstructions(context, components, utils)
488488

489489
FunctionExpression: handleFunctionLikeExpressions,
490490

491+
JSXSpreadAttribute: function(node) {
492+
const component = components.get(utils.getParentComponent());
493+
components.set(component ? component.node : node, {
494+
ignoreUnusedPropTypesValidation: true
495+
});
496+
},
497+
491498
MemberExpression: function(node) {
492499
if (isPropTypesUsage(node)) {
493500
markPropTypesAsUsed(node);

tests/lib/rules/prop-types.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3889,7 +3889,7 @@ ruleTester.run('prop-types', rule, {
38893889
`,
38903890
errors: [{
38913891
message: '\'foo.baz\' is missing in props validation'
3892-
}],
3892+
}]
38933893
},
38943894
{
38953895
code: `

0 commit comments

Comments
 (0)