Skip to content

Dependency updates (18 Feb 2026)#609

Open
1456055067 wants to merge 6 commits intotrishume:masterfrom
1456055067:deps-updates-18FEB2026
Open

Dependency updates (18 Feb 2026)#609
1456055067 wants to merge 6 commits intotrishume:masterfrom
1456055067:deps-updates-18FEB2026

Conversation

@1456055067
Copy link
Contributor

Summary

  • Bump yaml-rust2 from 0.10.4 to 0.11.0 (pulls in hashlink 0.11)
  • Bump fancy-regex from 0.16.2 to 0.17.0
  • Bump pretty_assertions from 0.6 to 1.4.1 (dev-dep, no code changes needed)
  • Bump criterion from 0.3 to 0.8.2 (dev-dep, rewrote bench files to use the modern closure-based API — Bencher as a standalone function parameter was removed in 0.4)
  • Fix pre-existing clippy::derive_ord_xor_partial_ord lint on MatchPower: replaced derive(PartialOrd) + explicit impl Ord with a manual PartialOrd that delegates to Ord via f64::total_cmp; updated public API snapshot accordingly

Verification

All of the following were run locally and passed:

  • cargo test — 129 tests passed (unit, integration, doc-tests, public API snapshot)
  • cargo fmt --check — no issues
  • cargo clippy — no warnings or errors
  • cargo build --benches — clean build
  • cargo bench — all benchmarks completed successfully

Benchmark results (vs. previous baseline)

Improved:

Benchmark Time Δ
highlight/"parser.rs" 136.5 ms −4.9%
highlight/"scope.rs" 15.2 ms −33.8%
load_and_highlight/"highlight_test.erb" 11.9 ms −25.0%
load_and_highlight/"InspiredGitHub.tmTheme" 9.2 ms −12.8%
add_from_folder 189 ms −2.7%
parse/"scope.rs" 12.8 ms −4.5%

Minor regression (within noise):

Benchmark Time Δ
from_dump_file 298 µs +2.2%

All other benchmarks showed no statistically significant change.

Replace derived PartialOrd (which conflicted with explicit Ord) with a
manual PartialOrd impl that delegates to Ord, using f64::total_cmp.
Update public API snapshot for the &Self -> &MatchPower sig change.
Rewrite bench files to drop the removed standalone Bencher parameter
style in favour of closures passed directly to bench_function/
bench_with_input, which is the modern criterion API.
@1456055067 1456055067 marked this pull request as ready for review February 18, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants