Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

feat: expose (utf8|binary)_substring #1539

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

orlp
Copy link
Contributor

@orlp orlp commented Aug 14, 2023

Closes #1538.

@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (36e905d) 83.45% compared to head (d9c4b38) 83.45%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1539   +/-   ##
=======================================
  Coverage   83.45%   83.45%           
=======================================
  Files         388      388           
  Lines       42138    42142    +4     
=======================================
+ Hits        35165    35171    +6     
+ Misses       6973     6971    -2     
Files Changed Coverage Δ
src/compute/substring.rs 100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

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

@ritchie46
Copy link
Collaborator

Can you run a cargo fmt?

@orlp
Copy link
Contributor Author

orlp commented Aug 14, 2023

Oops, sorry. Done.

@ritchie46 ritchie46 merged commit 86b8a5b into jorgecarleitao:main Aug 16, 2023
25 checks passed
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.

Expose utf8_substring and binary_substring
2 participants