Skip to content

Conversation

@robertpi
Copy link
Member

@robertpi robertpi commented Sep 3, 2025

The syntax of the raw string was incorrect and broke the regex.

@robertpi robertpi marked this pull request as ready for review September 3, 2025 17:47
@robertpi robertpi requested a review from a team as a code owner September 3, 2025 17:47
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.97%. Comparing base (1054161) to head (d2a08c6).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #251      +/-   ##
==========================================
+ Coverage   68.94%   68.97%   +0.02%     
==========================================
  Files          56       56              
  Lines        7307     7313       +6     
  Branches     1034     1034              
==========================================
+ Hits         5038     5044       +6     
  Misses       1769     1769              
  Partials      500      500              
Files with missing lines Coverage Δ
src/security/library.cpp 59.11% <100.00%> (+0.38%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cataphract cataphract merged commit 655c4d4 into master Sep 5, 2025
59 checks passed
@cataphract cataphract deleted the robert/update-regex branch September 5, 2025 13:54
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.

4 participants