Skip to content

Replace deprecated String.prototype.substr()#42

Merged
wesleytodd merged 1 commit intojshttp:masterfrom
CommanderRoot:rm-deprecated-substr
Jul 26, 2024
Merged

Replace deprecated String.prototype.substr()#42
wesleytodd merged 1 commit intojshttp:masterfrom
CommanderRoot:rm-deprecated-substr

Conversation

@CommanderRoot
Copy link
Contributor

String.prototype.substr() is deprecated so we replace it with String.prototype.slice() which works similarily but isn't deprecated.
.substr() probably isn't going away anytime soon but the change is trivial so it doesn't hurt to do it.

.substr() is deprecated so we replace it with .slice() which works similarily but isn't deprecated
Signed-off-by: Tobias Speicher <rootcommander@gmail.com>
@ejcheng ejcheng added the pr label Mar 24, 2022
@wesleytodd wesleytodd merged commit adab325 into jshttp:master Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants