Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

Conversation

@seer-by-sentry
Copy link
Contributor

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by Nora Shapiro.

Fixes WORKER-S1E. The issue was that: Invalid regex pattern * in branches setting from codecov.yaml caused re.match to fail without validation or transformation.

  • Introduces a _convert_glob_to_regex function to convert glob-style patterns to regular expressions.
  • Modifies the match function to use the converted regex patterns when matching strings, allowing for more flexible pattern matching using glob-style wildcards.

If you have any questions or feedback for the Sentry team about this fix, please email autofix@sentry.io with the Run ID: 29688.

@seer-by-sentry
Copy link
Contributor Author

✅ Sentry found no issues in your recent changes ✅

@nora-shap nora-shap closed this Apr 18, 2025
@codecov-notifications
Copy link

codecov-notifications bot commented Apr 18, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
helpers/match.py 87.50% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Apr 18, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 97.74%. Comparing base (7a25b06) to head (65168b4).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
helpers/match.py 87.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1216      +/-   ##
==========================================
- Coverage   97.74%   97.74%   -0.01%     
==========================================
  Files         459      459              
  Lines       37296    37302       +6     
==========================================
+ Hits        36454    36459       +5     
- Misses        842      843       +1     
Flag Coverage Δ
integration 42.72% <12.50%> (-0.01%) ⬇️
unit 90.80% <87.50%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant