Skip to content

IntelliSense doc string.replace typo #41385

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

Merged
merged 6 commits into from
May 25, 2021

Conversation

Jujhar
Copy link
Contributor

@Jujhar Jujhar commented Nov 3, 2020

Fixes #41361

As per https://www.w3schools.com/jsref/jsref_replace.asp
replace only replaces first value unless flags are used.

@typescript-bot
Copy link
Collaborator

It looks like you've sent a pull request to update our 'lib' files. These files aren't meant to be edited by hand, as they consist of last-known good states of the compiler and are generated from 'src'. Unless this is necessary, consider closing the pull request and sending a separate PR to update 'src'.

@typescript-bot typescript-bot added lib update PR modifies files in the `lib` folder For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Nov 3, 2020
@Jujhar

This comment has been minimized.

Copy link
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments on wording. Also:

  1. Still need to revert lib change.
  2. The linter is complaining about formatting =(

@sandersn sandersn self-assigned this Nov 18, 2020
@typescript-bot typescript-bot added For Backlog Bug PRs that fix a backlog bug and removed For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Nov 18, 2020
Copy link
Contributor

@orta orta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kinda surprised this doesn't break baselines, but it looks good to me

@Jujhar Jujhar requested a review from sandersn May 14, 2021 00:37
@sandersn
Copy link
Member

@orta I think this change would only show up in a quickinfo baseline that explicitly tests replace. I guess we don't have any of those.

@sandersn sandersn merged commit 52cefdf into microsoft:master May 25, 2021
typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request May 25, 2021
* Fix IntelliSense doc string.replace typo

* Update IntelliSense doc string.replace

* Update string.replace doc text

* Revert "Fix IntelliSense doc string.replace typo"

This reverts commit 4220ec8.

* Reworded string.replace doc text

* Fix doc string.replace text
typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request May 25, 2021
* Fix IntelliSense doc string.replace typo

* Update IntelliSense doc string.replace

* Update string.replace doc text

* Revert "Fix IntelliSense doc string.replace typo"

This reverts commit 4220ec8.

* Reworded string.replace doc text

* Fix doc string.replace text
typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request May 25, 2021
* Fix IntelliSense doc string.replace typo

* Update IntelliSense doc string.replace

* Update string.replace doc text

* Revert "Fix IntelliSense doc string.replace typo"

This reverts commit 4220ec8.

* Reworded string.replace doc text

* Fix doc string.replace text
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Backlog Bug PRs that fix a backlog bug lib update PR modifies files in the `lib` folder
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

string.replace IntelliSense is conveying wrong meaning
4 participants