Skip to content

Parse: minor optimizations#26

Merged
milosgajdos merged 1 commit into
distribution:mainfrom
thaJeztah:optimize_parse
Apr 18, 2026
Merged

Parse: minor optimizations#26
milosgajdos merged 1 commit into
distribution:mainfrom
thaJeztah:optimize_parse

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Apr 18, 2026

Copy link
Copy Markdown
Member
  • add early return for empty strings
  • skip lowercase-regex if the input is already lowercase
@thaJeztah
Parse: minor optimizations
2bae058 
- add early return for empty strings
- skip lowercase-regex if the input is already lowercase

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov

codecov Bot commented Apr 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.21%. Comparing base (c577ba6) to head (2bae058).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
+ Coverage   83.71%   84.21%   +0.49%     
==========================================
  Files           5        5              
  Lines         393      304      -89     
==========================================
- Hits          329      256      -73     
+ Misses         54       38      -16     
  Partials       10       10

☔ 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.
@thaJeztah thaJeztah mentioned this pull request Apr 18, 2026
Merged
1 task
@thaJeztah thaJeztah requested a review from milosgajdos April 18, 2026 09:45
@milosgajdos milosgajdos merged commit 56cff07 into distribution:main Apr 18, 2026
13 checks passed
@thaJeztah thaJeztah deleted the optimize_parse branch April 18, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@thaJeztah @milosgajdos