Skip to content

fix: replace panicking expect() with map_or() for edge-case file paths #2679

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

Open
srpatcha wants to merge 1 commit into
base: main
Choose a base branch
from
+24 −2

fix: replace panicking expect() with map_or() for file_name() check

9ed2e8a
Select commit
Loading
Failed to load commit list.
Open

fix: replace panicking expect() with map_or() for edge-case file paths #2679

fix: replace panicking expect() with map_or() for file_name() check
9ed2e8a
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Apr 18, 2026

Performance Gate Passed

✅ 60 untouched benchmarks
⏩ 4 skipped benchmarks1

Comparing srpatcha:fix/lru-cache-path-panic (9ed2e8a) with main (d11e2e0)

Open in CodSpeed

Footnotes

  1. 4 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

View more details on CodSpeed HQ