Skip to content

Commit 04964fd

Browse files
committed
Refactor code-style
1 parent fbe2714 commit 04964fd

File tree

5 files changed

+42
-34
lines changed

5 files changed

+42
-34
lines changed

.prettierignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
coverage/
2+
hast-util-whitespace.js
3+
hast-util-whitespace.min.js

index.js

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,22 @@
1-
'use strict';
1+
'use strict'
22

3-
/* Expose. */
4-
module.exports = interElementWhiteSpace;
3+
module.exports = interElementWhiteSpace
54

65
/* HTML white-space expression.
76
* See <https://html.spec.whatwg.org/#space-character>. */
8-
var re = /[ \t\n\f\r]/g;
7+
var re = /[ \t\n\f\r]/g
98

109
/* Check if `node` is a inter-element white-space. */
1110
function interElementWhiteSpace(node) {
12-
var value;
11+
var value
1312

1413
if (node && typeof node === 'object' && node.type === 'text') {
15-
value = node.value || '';
14+
value = node.value || ''
1615
} else if (typeof node === 'string') {
17-
value = node;
16+
value = node
1817
} else {
19-
return false;
18+
return false
2019
}
2120

22-
return value.replace(re, '') === '';
21+
return value.replace(re, '') === ''
2322
}

package.json

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,23 +26,31 @@
2626
"browserify": "^16.0.0",
2727
"esmangle": "^1.0.1",
2828
"nyc": "^12.0.0",
29+
"prettier": "^1.13.5",
2930
"remark-cli": "^5.0.0",
3031
"remark-preset-wooorm": "^4.0.0",
3132
"tape": "^4.4.0",
3233
"xo": "^0.21.0"
3334
},
3435
"scripts": {
35-
"build-md": "remark . --quiet --frail --output",
36+
"format": "remark . -qfo && prettier --write '**/*.js' && xo --fix",
3637
"build-bundle": "browserify index.js --bare -s hastUtilWhitespace > hast-util-whitespace.js",
3738
"build-mangle": "esmangle hast-util-whitespace.js > hast-util-whitespace.min.js",
38-
"build": "npm run build-md && npm run build-bundle && npm run build-mangle",
39-
"lint": "xo",
40-
"test-api": "node test.js",
39+
"build": "npm run build-bundle && npm run build-mangle",
40+
"test-api": "node test",
4141
"test-coverage": "nyc --reporter lcov tape test.js",
42-
"test": "npm run build && npm run lint && npm run test-coverage"
42+
"test": "npm run format && npm build lint && npm run test-coverage"
43+
},
44+
"prettier": {
45+
"tabWidth": 2,
46+
"useTabs": false,
47+
"singleQuote": true,
48+
"bracketSpacing": false,
49+
"semi": false,
50+
"trailingComma": "none"
4351
},
4452
"xo": {
45-
"space": true,
53+
"prettier": true,
4654
"esnext": false,
4755
"ignores": [
4856
"hast-util-is-element.js"

readme.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,23 @@ npm install hast-util-whitespace
1414
## Usage
1515

1616
```javascript
17-
var whitespace = require('hast-util-whitespace');
17+
var whitespace = require('hast-util-whitespace')
1818

1919
whitespace({
2020
type: 'element',
2121
tagName: 'div',
2222
children: []
23-
}); //=> false
23+
}) // => false
2424

2525
whitespace({
2626
type: 'text',
2727
value: '\t \n'
28-
}); //=> true
28+
}) // => true
2929

3030
whitespace({
3131
type: 'text',
3232
value: ' text\f'
33-
}); //=> false
33+
}) // => false
3434
```
3535

3636
## API

test.js

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,46 @@
1-
'use strict';
1+
'use strict'
22

3-
/* Dependencies. */
4-
var test = require('tape');
5-
var whitespace = require('.');
3+
var test = require('tape')
4+
var whitespace = require('.')
65

7-
/* Tests. */
8-
test('whitespace', function (t) {
9-
t.equal(whitespace(), false, 'should return `false` without node');
6+
test('whitespace', function(t) {
7+
t.equal(whitespace(), false, 'should return `false` without node')
108

119
t.equal(
1210
whitespace({type: 'element', tagName: 'div'}),
1311
false,
1412
'should return `false` without text'
15-
);
13+
)
1614

1715
t.equal(
1816
whitespace({type: 'text', value: '\v'}),
1917
false,
2018
'should return `false` for other white-space'
21-
);
19+
)
2220

2321
t.equal(
2422
whitespace({type: 'text', value: ' \t\r\n\f'}),
2523
true,
2624
'should return `true` for inter-element white-space'
27-
);
25+
)
2826

2927
t.equal(
3028
whitespace({type: 'text'}),
3129
true,
3230
'should return `true` for `text` without value'
33-
);
31+
)
3432

3533
t.equal(
3634
whitespace(' \v'),
3735
false,
3836
'should return `false` for a `string` of text'
39-
);
37+
)
4038

4139
t.equal(
4240
whitespace(' \t\r\n\f'),
4341
true,
4442
'should return `true` for a `string` of inter-element white-space'
45-
);
43+
)
4644

47-
t.end();
48-
});
45+
t.end()
46+
})

0 commit comments

Comments
 (0)