Skip to content

perf: do prettification after deduping #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Oct 24, 2025
Merged

perf: do prettification after deduping #18

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e27f04b
perf: do prettification after deduping
bartveneman Oct 20, 2025
328afb4
sort ranges
bartveneman Oct 20, 2025
47bb65d
closing brace can be covered, np
bartveneman Oct 20, 2025
f56875c
it works now!!!
bartveneman Oct 22, 2025
54bd4de
more amazing work done
bartveneman Oct 22, 2025
ddc31f9
more tests and fixes
bartveneman Oct 22, 2025
1a55306
more fixes
bartveneman Oct 22, 2025
c32bea8
soooo close now
bartveneman Oct 24, 2025
13c34e8
clean up and skip cli for now
bartveneman Oct 24, 2025
31d5c3c
add the biggest test suite possible
bartveneman Oct 24, 2025
1ff3e8d
cleaning up
bartveneman Oct 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
closing brace can be covered, np
  • Loading branch information
@bartveneman
bartveneman committed Oct 22, 2025
commit 47bb65d7d64c55b53578614e72d3797be31feed4
2 changes: 1 addition & 1 deletion src/lib/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ function calculate_stylesheet_coverage({ text, url, ranges }: Coverage) {

let prev_is_covered = index > 0 && line_coverage[index - 1] === 1

if (is_in_range && !is_closing_brace && !is_empty) {
if (is_in_range) {
is_covered = true
} else if ((is_empty || is_closing_brace) && prev_is_covered) {
is_covered = true
Expand Down