Skip to content

$all #460

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.

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 1 commit into from
May 16, 2025
Merged

$all #460

merged 1 commit into from
May 16, 2025

Conversation

antonok-edm
Copy link
Collaborator

This won't include popup, inline-script, or inline-font for now due to lack of support for the individual options, but we may as well convert $all into all supported types.

partially addresses #1

@antonok-edm antonok-edm self-assigned this May 16, 2025
@antonok-edm antonok-edm added the enhancement New feature or request label May 16, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rust Benchmark

Benchmark suite Current: 5459095 Previous: d56be21 Ratio
rule-match-browserlike/brave-list 1874774229 ns/iter (± 10126118) 1727033355 ns/iter (± 11920489) 1.09
rule-match-first-request/brave-list 999564 ns/iter (± 6431) 1005963 ns/iter (± 12764) 0.99
blocker_new/brave-list 210449159 ns/iter (± 1270776) 220140757 ns/iter (± 5159081) 0.96
memory-usage/brave-list-initial 41763292 ns/iter (± 3) 41408849 ns/iter (± 3) 1.01
memory-usage/brave-list-after-1000-requests 44356820 ns/iter (± 3) 44004875 ns/iter (± 3) 1.01

This comment was automatically generated by workflow using github-action-benchmark.

this won't include `popup`, `inline-script`, or `inline-font` for now
due to lack of support for the individual options, but we may as well
convert `$all` into all supported types.
@antonok-edm antonok-edm merged commit e0076e3 into v0.9.x May 16, 2025
6 checks passed
@antonok-edm antonok-edm deleted the all branch May 16, 2025 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant