Skip to content

Commit 7425244

Browse files
committed
refactor: avoid duplicate implementation for isJSXText
1 parent 01eb97e commit 7425244

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

lib/rules/jsx-no-useless-fragment.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ const pragmaUtil = require('../util/pragma');
77
const jsxUtil = require('../util/jsx');
88
const docsUrl = require('../util/docsUrl');
99

10+
function isJSXText(node) {
11+
return !!node && (node.type === 'JSXText' || node.type === 'Literal');
12+
}
1013

1114
module.exports = {
1215
meta: {
@@ -34,7 +37,7 @@ module.exports = {
3437
* @returns {boolean}
3538
*/
3639
function isPaddingSpaces(node) {
37-
return (node.type === 'JSXText' || node.type === 'Literal')
40+
return isJSXText(node)
3841
&& /^\s*$/.test(node.raw)
3942
&& node.raw.includes('\n');
4043
}
@@ -65,10 +68,6 @@ module.exports = {
6568
&& /^[a-z]+$/.test(node.parent.openingElement.name.name);
6669
}
6770

68-
function isJSXText(node) {
69-
return !!node && (node.type === 'JSXText' || node.type === 'Literal');
70-
}
71-
7271
/**
7372
* Avoid fixing case like:
7473
* ```jsx

0 commit comments

Comments
 (0)