Skip to content

fix: handle escaped exact paths in baseline/relax#48

Merged
jakekaplan merged 2 commits intomainfrom
fix-46-escaped-glob-roundtrip
Feb 10, 2026
Merged

fix: handle escaped exact paths in baseline/relax#48
jakekaplan merged 2 commits intomainfrom
fix-46-escaped-glob-roundtrip

Conversation

@jakekaplan
Copy link
Copy Markdown
Owner

Summary

  • switch exact-path detection to unescape/escape round-trip logic
  • escape exact paths when writing baseline/relax rules
  • unescape stored paths before matching existing exact-path rules
  • add regression coverage for escaped paths in unit + integration tests

Fixes #46

Use escape/unescape round-trip for exact-path detection, store escaped literal paths, and add escaped path regression tests for baseline and relax.

Fixes #46
Copy link
Copy Markdown

@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.

Benchmark

Details
Benchmark suite Current: 7da4328 Previous: 76f6472 Ratio
cpython 0.05151774096 seconds (± 0.0013) 0.04199398728000001 seconds (± 0.0044) 1.23
airflow 0.14225667106 seconds (± 0.0008) 0.10725717758000002 seconds (± 0.0005) 1.33
prefect 0.06045304346 seconds (± 0.0004) 0.04777859738000001 seconds (± 0.0019) 1.27
ruff 0.12567846496 seconds (± 0.0012) 0.09690332188 seconds (± 0.0005) 1.30

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

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.63%. Comparing base (76f6472) to head (7da4328).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   98.51%   98.63%   +0.11%     
==========================================
  Files          21       21              
  Lines        3175     3225      +50     
==========================================
+ Hits         3128     3181      +53     
+ Misses         47       44       -3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jakekaplan jakekaplan merged commit 3065f71 into main Feb 10, 2026
9 checks passed
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.

loq baseline should escape brackets in paths

1 participant