Skip to content

perf: array.includes → set.has#14

Merged
Kikobeats merged 5 commits intomasterfrom
perf
Jan 4, 2025
Merged

perf: array.includes → set.has#14
Kikobeats merged 5 commits intomasterfrom
perf

Conversation

@Kikobeats
Copy link
Copy Markdown
Owner

@Kikobeats Kikobeats commented Jan 4, 2025 •
edited
Loading 

node sort-headers.mjs
┌─────────┬─────────────────┬──────────────────────┬─────────────────────┬────────────────────────────┬───────────────────────────┬─────────┐
│ (index) │ Task name       │ Latency average (ns) │ Latency median (ns) │ Throughput average (ops/s) │ Throughput median (ops/s) │ Samples │
├─────────┼─────────────────┼──────────────────────┼─────────────────────┼────────────────────────────┼───────────────────────────┼─────────┤
│ 0       │ 'v0 (for loop)' │ '4414.58 ± 0.33%'    │ '4333.00'           │ '230104 ± 0.02%'           │ '230787'                  │ 226523  │
│ 1       │ 'v1 (set)'      │ '3641.63 ± 0.16%'    │ '3583.00'           │ '276968 ± 0.01%'           │ '279096'                  │ 274602  │
└─────────┴─────────────────┴──────────────────────┴─────────────────────┴────────────────────────────┴───────────────────────────┴─────────┘

@Kikobeats Kikobeats changed the title test: add user-agent cases perf: array.includes → set.has Jan 4, 2025
@Kikobeats Kikobeats merged commit d1062a6 into master Jan 4, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Jan 4, 2025

Pull Request Test Coverage Report for Build 12614385664

Details

  • 19 of 19 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.5%) to 98.175%
Totals Coverage Status
Change from base Build 12601346237: 0.5%
Covered Lines: 244
Relevant Lines: 246
💛 - Coveralls

@Kikobeats Kikobeats deleted the perf branch January 4, 2025 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kikobeats @coveralls