-
-
Notifications
You must be signed in to change notification settings - Fork 113
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(stdlib): Add replacement functions to String module #1441
Conversation
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.
Looks pretty good! Just a couple of tweaks and this is perfect code-wise 🙂
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.
Just a few more updates to the comments, but otherwise looks great :)
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Applied Those Changes |
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 @spotandjake!
Co-authored-by: Blaine Bublitz <blaine.bublitz@gmail.com>
Applied That Change |
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.
Needs some docs work and you added a file that shouldn't be here. Please be careful when committing!
Applied Those Suggestions |
79a98f4
to
dc55f41
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.
Looks great! Thanks @spotandjake
This pr implements
String.replaceFirst
andString.replaceLast
andString.replaceAll
. let me know if you guys want the naming scheme to be changed.