• perf: remove Result from CachedPathImpl::canonicalized #847 👈 (View in Graphite)
• main

How to use the Graphite Merge Queue

Add the label merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.87%. Comparing base (f513bbd) to head (01d6be7).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   93.90%   93.87%   -0.04%     
==========================================
  Files          17       17              
  Lines        3086     3084       -2     
==========================================
- Hits         2898     2895       -3     
- Misses        188      189       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

CodSpeed Performance Report

Merging #847 will improve performances by 3.79%

_{Comparing 11-18-perf_remove_result_from_cachedpathimpl_canonicalized_ (01d6be7) with main (f513bbd)}

Summary

⚡ 1 improvement
✅ 10 untouched
⏩ 5 skipped¹

Benchmarks breakdown