Skip to content

Commit ce4336f

Browse files
committed
refactor: avoid duplicate implementation for isJSXText
1 parent 4f30022 commit ce4336f

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
@@ -8,6 +8,9 @@ const pragmaUtil = require('../util/pragma');
88
const jsxUtil = require('../util/jsx');
99
const docsUrl = require('../util/docsUrl');
1010

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

1215
module.exports = {
1316
meta: {
@@ -35,7 +38,7 @@ module.exports = {
3538
* @returns {boolean}
3639
*/
3740
function isPaddingSpaces(node) {
38-
return (node.type === 'JSXText' || node.type === 'Literal') &&
41+
return isJSXText(node) &&
3942
/^\s*$/.test(node.raw) &&
4043
node.raw.includes('\n');
4144
}
@@ -66,10 +69,6 @@ module.exports = {
6669
/^[a-z]+$/.test(node.parent.openingElement.name.name);
6770
}
6871

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

0 commit comments

Comments
 (0)