-
Notifications
You must be signed in to change notification settings - Fork 118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT] Add find functionality for string #2046
Conversation
861f6ed
to
f1f1e30
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work so far! Few comments/questions.
6275ae0
to
d50d161
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Just two comments left and then we should be good to go.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2046 +/- ##
=======================================
Coverage 84.72% 84.72%
=======================================
Files 62 62
Lines 6840 6849 +9
=======================================
+ Hits 5795 5803 +8
- Misses 1045 1046 +1
|
Great work! 🚀 |
Resolves #1925
Returns the first occurrence of the
substr
provided,-1
, in case of fails to find.