Skip to content

Commit 020648c

Browse files
romainmenkeromainmenke
committed
lint rules for braces and indent
1 parent 9c09b71 commit 020648c

File tree

21 files changed

+179
-136
lines changed

21 files changed

+179
-136
lines changed

‎.eslintrc.json

+4-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,10 @@
1111
"rules": {
1212
"quotes": ["error", "single"],
1313
"comma-dangle": ["error", "always-multiline"],
14-
"semi": ["error", "always"]
14+
"semi": ["error", "always"],
15+
"curly": "error",
16+
"brace-style": "error",
17+
"indent": ["error", "tab", { "SwitchCase": 1 }]
1518
},
1619
"overrides": [
1720
{

‎cli/csstools-cli/src/cli.ts

+35-35
Original file line numberDiff line numberDiff line change
@@ -87,44 +87,44 @@ function main() {
8787
postcssPseudoClassAnyLink();
8888
return;
8989
default:
90-
{
91-
const allHelp = [
92-
'CSSTools CLI\n',
93-
' Transform CSS with any plugin from https://github.com/csstools/postcss-plugins\n',
90+
{
91+
const allHelp = [
92+
'CSSTools CLI\n',
93+
' Transform CSS with any plugin from https://github.com/csstools/postcss-plugins\n',
9494

95-
'Usage:',
96-
' @csstools/csstools-cli postcss-preset-env [input.css] [OPTIONS] [-o|--output output.css]',
97-
' @csstools/csstools-cli postcss-preset-env <input.css>... [OPTIONS] --dir <output-directory>',
98-
' @csstools/csstools-cli postcss-preset-env <input.css>... [OPTIONS] --replace',
95+
'Usage:',
96+
' @csstools/csstools-cli postcss-preset-env [input.css] [OPTIONS] [-o|--output output.css]',
97+
' @csstools/csstools-cli postcss-preset-env <input.css>... [OPTIONS] --dir <output-directory>',
98+
' @csstools/csstools-cli postcss-preset-env <input.css>... [OPTIONS] --replace',
9999

100-
'\nAvailable Plugins:',
101-
' css-blank-pseudo',
102-
' css-has-pseudo',
103-
' css-prefers-color-scheme',
104-
' postcss-color-functional-notation',
105-
' postcss-color-hex-alpha',
106-
' postcss-color-rebeccapurple',
107-
' postcss-custom-properties',
108-
' postcss-dir-pseudo-class',
109-
' postcss-double-position-gradients',
110-
' postcss-env-function',
111-
' postcss-focus-visible',
112-
' postcss-focus-within',
113-
' postcss-gap-properties',
114-
' postcss-image-set-function',
115-
' postcss-lab-function',
116-
' postcss-logical',
117-
' postcss-nesting',
118-
' postcss-overflow-shorthand',
119-
' postcss-place',
120-
' postcss-pseudo-class-any-link',
100+
'\nAvailable Plugins:',
101+
' css-blank-pseudo',
102+
' css-has-pseudo',
103+
' css-prefers-color-scheme',
104+
' postcss-color-functional-notation',
105+
' postcss-color-hex-alpha',
106+
' postcss-color-rebeccapurple',
107+
' postcss-custom-properties',
108+
' postcss-dir-pseudo-class',
109+
' postcss-double-position-gradients',
110+
' postcss-env-function',
111+
' postcss-focus-visible',
112+
' postcss-focus-within',
113+
' postcss-gap-properties',
114+
' postcss-image-set-function',
115+
' postcss-lab-function',
116+
' postcss-logical',
117+
' postcss-nesting',
118+
' postcss-overflow-shorthand',
119+
' postcss-place',
120+
' postcss-pseudo-class-any-link',
121121

122-
'\nPlugin Help:',
123-
' @csstools/csstools-cli <plugin-name>',
124-
' @csstools/csstools-cli postcss-preset-env',
125-
];
122+
'\nPlugin Help:',
123+
' @csstools/csstools-cli <plugin-name>',
124+
' @csstools/csstools-cli postcss-preset-env',
125+
];
126126

127-
console.warn(allHelp.join('\n'));
128-
}
127+
console.warn(allHelp.join('\n'));
128+
}
129129
}
130130
}

‎experimental/css-has-pseudo/src/index.js

+35-35
Original file line numberDiff line numberDiff line change
@@ -130,42 +130,42 @@ function selectorSpecificity(node) {
130130
case ':is':
131131
case ':has':
132132
case ':not':
133-
{
134-
if (node.nodes && node.nodes.length > 0) {
135-
let mostSpecificListItem = {
136-
a: 0,
137-
b: 0,
138-
c: 0,
139-
};
140-
141-
node.nodes.forEach((child) => {
142-
const itemSpecificity = selectorSpecificity(child);
143-
if (itemSpecificity.a > mostSpecificListItem.a) {
144-
mostSpecificListItem = itemSpecificity;
145-
return;
146-
} else if (itemSpecificity.a < mostSpecificListItem.a) {
147-
return;
148-
}
149-
150-
if (itemSpecificity.b > mostSpecificListItem.b) {
151-
mostSpecificListItem = itemSpecificity;
152-
return;
153-
} else if (itemSpecificity.b < mostSpecificListItem.b) {
154-
return;
155-
}
156-
157-
if (itemSpecificity.c > mostSpecificListItem.c) {
158-
mostSpecificListItem = itemSpecificity;
159-
return;
160-
}
161-
});
162-
163-
a += mostSpecificListItem.a;
164-
b += mostSpecificListItem.b;
165-
c += mostSpecificListItem.c;
166-
}
167-
break;
133+
{
134+
if (node.nodes && node.nodes.length > 0) {
135+
let mostSpecificListItem = {
136+
a: 0,
137+
b: 0,
138+
c: 0,
139+
};
140+
141+
node.nodes.forEach((child) => {
142+
const itemSpecificity = selectorSpecificity(child);
143+
if (itemSpecificity.a > mostSpecificListItem.a) {
144+
mostSpecificListItem = itemSpecificity;
145+
return;
146+
} else if (itemSpecificity.a < mostSpecificListItem.a) {
147+
return;
148+
}
149+
150+
if (itemSpecificity.b > mostSpecificListItem.b) {
151+
mostSpecificListItem = itemSpecificity;
152+
return;
153+
} else if (itemSpecificity.b < mostSpecificListItem.b) {
154+
return;
155+
}
156+
157+
if (itemSpecificity.c > mostSpecificListItem.c) {
158+
mostSpecificListItem = itemSpecificity;
159+
return;
160+
}
161+
});
162+
163+
a += mostSpecificListItem.a;
164+
b += mostSpecificListItem.b;
165+
c += mostSpecificListItem.c;
168166
}
167+
break;
168+
}
169169

170170
case 'where':
171171
break;

‎plugin-packs/postcss-preset-env/src/lib/plugin-id-help.js

+10-6
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,12 @@ function mostSimilar(a, b) {
4242
}
4343

4444
function levenshteinDistance(s, t) {
45-
if (!s.length) return t.length;
46-
if (!t.length) return s.length;
45+
if (!s.length) {
46+
return t.length;
47+
}
48+
if (!t.length) {
49+
return s.length;
50+
}
4751
const arr = [];
4852
for (let i = 0; i <= t.length; i++) {
4953
arr[i] = [i];
@@ -52,10 +56,10 @@ function levenshteinDistance(s, t) {
5256
i === 0
5357
? j
5458
: Math.min(
55-
arr[i - 1][j] + 1,
56-
arr[i][j - 1] + 1,
57-
arr[i - 1][j - 1] + (s[j - 1] === t[i - 1] ? 0 : 1),
58-
);
59+
arr[i - 1][j] + 1,
60+
arr[i][j - 1] + 1,
61+
arr[i - 1][j - 1] + (s[j - 1] === t[i - 1] ? 0 : 1),
62+
);
5963
}
6064
}
6165
return arr[t.length][s.length];

‎plugins/postcss-color-hex-alpha/src/onCSSDeclaration.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,11 @@ const onCSSDeclaration = (decl) => {
1818
const modifiedValue = valueAST.toString();
1919

2020
if (modifiedValue !== originalValue) {
21-
if (options.preserve) decl.cloneBefore({ value: modifiedValue });
22-
else decl.value = modifiedValue;
21+
if (options.preserve) {
22+
decl.cloneBefore({ value: modifiedValue });
23+
} else {
24+
decl.value = modifiedValue;
25+
}
2326
}
2427
}
2528
};

‎plugins/postcss-color-rebeccapurple/src/onCSSDeclaration.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,11 @@ const onCSSDeclaration = decl => {
1818
const modifiedValue = String(valueAST);
1919

2020
if (modifiedValue !== originalValue) {
21-
if (options.preserve) decl.cloneBefore({ value: modifiedValue });
22-
else decl.value = modifiedValue;
21+
if (options.preserve) {
22+
decl.cloneBefore({ value: modifiedValue });
23+
} else {
24+
decl.value = modifiedValue;
25+
}
2326
}
2427
}
2528
};

‎plugins/postcss-color-rebeccapurple/src/onCSSIdentifier.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
/** @type {(decl: CSSIdentifier) => void} Transform the rebeccapurple keyword. */
22
const onCSSIdentifier = node => {
3-
if (isRebeccaPurple(node.value)) node.value = '#639';
3+
if (isRebeccaPurple(node.value)) {
4+
node.value = '#639';
5+
}
46
};
57

68
export default onCSSIdentifier;

‎plugins/postcss-custom-properties/src/lib/get-custom-properties-from-root.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ export default function getCustomPropertiesFromRoot(root, opts) {
1111
root.nodes.slice().forEach(rule => {
1212
const customPropertiesObject = isHtmlRule(rule)
1313
? customPropertiesFromHtmlElement
14-
: isRootRule(rule)
15-
? customPropertiesFromRootPseudo
16-
: null;
14+
: isRootRule(rule)
15+
? customPropertiesFromRootPseudo
16+
: null;
1717

1818
// for each custom property
1919
if (customPropertiesObject) {

‎plugins/postcss-lab-function/src/color.ts

+5-3
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,11 @@ export function labToSRgb(lab: [number, number, number]): [number, number, numbe
88

99
export function lchToSRgb(lch: [number, number, number]): [number, number, number] {
1010
return xyz2rgb(
11-
lab2xyz(lch2lab(lch),
12-
false, /* do not clamp "a" and "b" when processing lch */
13-
));
11+
lab2xyz(
12+
lch2lab(lch),
13+
false, /* do not clamp "a" and "b" when processing lch */
14+
),
15+
);
1416
}
1517

1618
function lch2lab(lch: [number, number, number]): [number, number, number] {

‎plugins/postcss-logical/src/lib/transform-border-radius.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,7 @@ function rDecl(decl) {
3535
}
3636

3737
function clean(decl, preserve) {
38-
if (!preserve) decl.remove();
38+
if (!preserve) {
39+
decl.remove();
40+
}
3941
}

‎plugins/postcss-logical/src/lib/transform-border.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -139,5 +139,7 @@ export default {
139139
};
140140

141141
function clean(decl, preserve) {
142-
if (!preserve) decl.remove();
142+
if (!preserve) {
143+
decl.remove();
144+
}
143145
}

‎plugins/postcss-logical/src/lib/transform-directional-shorthands.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -66,5 +66,7 @@ export default (decl, values, dir, preserve) => {
6666
};
6767

6868
function clean(decl, preserve) {
69-
if (!preserve) decl.remove();
69+
if (!preserve) {
70+
decl.remove();
71+
}
7072
}

‎plugins/postcss-logical/src/lib/transform-float.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -43,5 +43,7 @@ function rDecl(decl) {
4343
}
4444

4545
function clean(decl, preserve) {
46-
if (!preserve) decl.remove();
46+
if (!preserve) {
47+
decl.remove();
48+
}
4749
}

‎plugins/postcss-logical/src/lib/transform-inset.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -47,5 +47,7 @@ function rDecl(decl, values) {
4747
}
4848

4949
function clean(decl, preserve) {
50-
if (!preserve) decl.remove();
50+
if (!preserve) {
51+
decl.remove();
52+
}
5153
}

‎plugins/postcss-logical/src/lib/transform-resize.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,7 @@ export default (decl, values, dir, preserve) => {
1111
};
1212

1313
function clean(decl, preserve) {
14-
if (!preserve) decl.remove();
14+
if (!preserve) {
15+
decl.remove();
16+
}
1517
}

‎plugins/postcss-logical/src/lib/transform-side.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -109,5 +109,7 @@ export default {
109109
};
110110

111111
function clean(decl, preserve) {
112-
if (!preserve) decl.remove();
112+
if (!preserve) {
113+
decl.remove();
114+
}
113115
}

‎plugins/postcss-logical/src/lib/transform-text-align.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -43,5 +43,7 @@ function rDecl(decl) {
4343
}
4444

4545
function clean(decl, preserve) {
46-
if (!preserve) decl.remove();
46+
if (!preserve) {
47+
decl.remove();
48+
}
4749
}

‎plugins/postcss-logical/src/lib/transform-transition.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,9 @@ export default (decl, notValues, dir, preserve) => {
6666
};
6767

6868
function clean(decl, preserve) {
69-
if (!preserve) decl.remove();
69+
if (!preserve) {
70+
decl.remove();
71+
}
7072
}
7173

7274
const valueMap = {

‎plugins/postcss-nesting/src/lib/list.js

+9-3
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,19 @@ export const comma = (string) => {
2121
} else if (letter === '(') {
2222
func += 1;
2323
} else if (letter === ')') {
24-
if (func > 0) func -= 1;
24+
if (func > 0) {
25+
func -= 1;
26+
}
2527
} else if (func === 0) {
26-
if (letter === ',') split = true;
28+
if (letter === ',') {
29+
split = true;
30+
}
2731
}
2832

2933
if (split) {
30-
if (current !== '') array.push(current.trim());
34+
if (current !== '') {
35+
array.push(current.trim());
36+
}
3137
current = '';
3238
split = false;
3339
} else {

0 commit comments

Comments
 (0)