Skip to content

Ignore . prefixed directory when searching managed Python for pygrep#919

Merged
j178 merged 1 commit intomasterfrom
pygrep-python
Oct 18, 2025
Merged

Ignore . prefixed directory when searching managed Python for pygrep#919
j178 merged 1 commit intomasterfrom
pygrep-python

Conversation

@j178
Copy link
Owner

@j178 j178 commented Oct 18, 2025

For #918

@j178 j178 added the bug Something isn't working label Oct 18, 2025
@j178 j178 changed the title Ignore any . prefixed directories in managed Python directory Ignore . prefixed directory when searching managed Python for pygrep Oct 18, 2025
@codecov
Copy link

codecov bot commented Oct 18, 2025

Codecov Report

❌ Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.06%. Comparing base (d174e59) to head (0a0f4ad).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/languages/pygrep/pygrep.rs 16.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #919      +/-   ##
==========================================
- Coverage   90.10%   90.06%   -0.04%     
==========================================
  Files          64       64              
  Lines       11944    11950       +6     
==========================================
+ Hits        10762    10763       +1     
- Misses       1182     1187       +5     

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

@j178 j178 marked this pull request as ready for review October 18, 2025 12:00
@j178 j178 merged commit f13e392 into master Oct 18, 2025
18 checks passed
@j178 j178 deleted the pygrep-python branch October 18, 2025 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant