coreui / coreui-pro / 32491035323

93%

main: 94%

push

github

chore(Stylelint): align the config with upstream (#831)

Two rules upstream enforces and we did not, plus their file format so a rule
can carry the reason it exists.

`selector-not-notation: complex` is off in the shared config with a "TODO
enable this later". It is what catches a chained `:not(a):not(b)` — the thing
I have been fixing by hand after diffing files against upstream, twice in the
last week. Twenty-three selectors were still on the old form.

`selector-class-pattern` gains an optional prefix group. Without it a `.dark\:`
selector fails the kebab-case check, which left the decision to keep that one
name unusable in our own stylesheets. Responsive variants stay infixed, so the
group admits nothing else today.

Not taken: `order/order`. Upstream runs it, and applying it reordered 130
lines of `_root.scss` for four violations — which cost 0.08 kB on
`coreui-reboot.min.css`, 0.11 kB on `coreui-utilities.css` and 0.06 kB on its
minified twin, enough to breach three budgets. Identical bytes, worse
compression. Not worth three budget bumps for declaration order.

`calc` stays on `function-disallowed-list` for now — upstream dropped it, and
following them costs 148 disable comments across 56 files, which deserves its
own diff.

4537 of 5095 branches covered (89.05%)

Branch coverage included in aggregate %.

8661 of 9055 relevant lines covered (95.65%)

399.88 hits per line