feat: add string/base/replace-after

[AuenKr]

Resolves #812 .

Description

What is the purpose of this pull request?

This pull request:

• adding a utility to replace the substring after the first occurrence of a specified search string

Related Issues

Does this pull request have any related issues?

This pull request:

• resolves [RFC]: Add @stdlib/string/base/replace-after #812

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

• Read, understood, and followed the contributing guidelines.

---

@stdlib-js/reviewers

[stdlib-bot]

👋 Hi there! 👋

And thank you for opening your first pull request! We will review it shortly. 🏃 💨

[kgryte]

@AuenKr Would you mind updating the copyright years to 2024? Thanks!

[kgryte]

@AuenKr Overall, this PR looks great! Thanks for working on this!

[kgryte]

@AuenKr There's actually one change to this PR that we should make and that is support for fromIndex argument. This was previously discussed in https://github.com/stdlib-js/stdlib/pull/922/files.

The argument should be mandatory (unlike in that discussion), so the expected signature is

function replaceAfter( str, search, replacement, fromIndex ) {...}

Is this something you can include in this PR?

[kgryte]

@AuenKr This looks good. You just need to fix the broken tests, and once finished, this should be ready for merge.

[kgryte]

LGTM. Thanks, @AuenKr, and thank you for authoring a clean PR! Will merge once CI successfully completes.

[AuenKr]

@kgryte, Shouldn't string/base/replace-before also have fromIndex arguement??

[kgryte]

@AuenKr Yes, it should.