Skip to content

Bump regex from 1.7 to 1.10#391

Merged
sylvestre merged 1 commit into
uutils:mainfrom
cakebaker:bump_regex
Jun 24, 2024
Merged

Bump regex from 1.7 to 1.10#391
sylvestre merged 1 commit into
uutils:mainfrom
cakebaker:bump_regex

Conversation

@cakebaker
Copy link
Copy Markdown
Contributor

This PR bumps regex from 1.7 to 1.10 and as a result byteorder is no longer needed.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.74%. Comparing base (5b19ea2) to head (916313e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
+ Coverage   60.48%   64.74%   +4.26%     
==========================================
  Files          32       32              
  Lines        4069     3812     -257     
  Branches      895      875      -20     
==========================================
+ Hits         2461     2468       +7     
+ Misses       1254      993     -261     
+ Partials      354      351       -3     

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

@cakebaker
Copy link
Copy Markdown
Contributor Author

Changes since last push:

  • rebased
  • updated regex, regex-automata, and regex-syntax to the most recent versions in Cargo.lock

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Run GNU findutils tests: GNU tests summary = TOTAL: 718 / PASS: 433 / FAIL: 282 / ERROR: 2
Run GNU findutils tests: Changes from main: PASS +0 / FAIL +0 / ERROR +0 / SKIP +0 
Run BFS tests: Changes from main: PASS +0 / SKIP +0 / FAIL +0
Run BFS tests: BFS tests summary = TOTAL: 288 / PASS: 182 / SKIP: 1 / FAIL: 105

@sylvestre sylvestre merged commit 57b6706 into uutils:main Jun 24, 2024
@cakebaker cakebaker deleted the bump_regex branch June 24, 2024 06:20
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.

2 participants