Skip to content

fix: ignore baseline temp config file during scan#50

Merged
jakekaplan merged 1 commit intomainfrom
fix/baseline-ignore-temp-config
Feb 15, 2026
Merged

fix: ignore baseline temp config file during scan#50
jakekaplan merged 1 commit intomainfrom
fix/baseline-ignore-temp-config

Conversation

@jakekaplan
Copy link
Copy Markdown
Owner

baseline creates a temporary config file in the repo root while scanning violations. That temporary file could be scanned and baselined like a real source file, which produced bogus rules for .tmp* paths.

This change filters the temporary config file out of scan outcomes before building baseline violations. It also adds a regression test to lock behavior.

Impact: baseline only writes rules for actual project files.

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: 1ac6c7d Previous: 3065f71 Ratio
cpython 0.05542575824 seconds (± 0.0005) 0.043872018400000004 seconds (± 0.001) 1.26
airflow 0.15721443984000003 seconds (± 0.0032) 0.11329012640000002 seconds (± 0.0015) 1.39
prefect 0.06570204244 seconds (± 0.0008) 0.0511819063 seconds (± 0.0017) 1.28
ruff 0.13541981834 seconds (± 0.0011) 0.10253594740000001 seconds (± 0.0015) 1.32

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

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.64%. Comparing base (3065f71) to head (1ac6c7d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   98.63%   98.64%           
=======================================
  Files          21       21           
  Lines        3225     3242   +17     
=======================================
+ Hits         3181     3198   +17     
  Misses         44       44           

☔ 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 b97a7fa into main Feb 15, 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.

1 participant