Skip to content

replace[Regexp]{One,All}(): allow non-const pattern and replacement arguments#46589

Merged
CurtizJ merged 2 commits intomasterfrom
rs/nonconst-replace
Apr 6, 2023
Merged

replace[Regexp]{One,All}(): allow non-const pattern and replacement arguments#46589
CurtizJ merged 2 commits intomasterfrom
rs/nonconst-replace

Conversation

@rschu1ze
Copy link
Copy Markdown
Member

@rschu1ze rschu1ze commented Feb 20, 2023

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Functions replaceOne(), replaceAll(), replaceRegexpOne() and replaceRegexpAll() can now be called with non-const pattern and replacement arguments

@rschu1ze rschu1ze added the do not test disable testing on pull request label Feb 20, 2023
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-improvement Pull request with some product improvements label Feb 20, 2023
@rschu1ze rschu1ze force-pushed the rs/nonconst-replace branch from 3689fb7 to 147505e Compare February 20, 2023 11:08
@rschu1ze rschu1ze changed the title (wip) replace*(): allow non-const pattern and replacement arguments (wip) replace{One,All}(): allow non-const pattern and replacement arguments Feb 20, 2023
@rschu1ze rschu1ze force-pushed the rs/nonconst-replace branch from 147505e to d39dbe8 Compare February 20, 2023 19:06
@rschu1ze rschu1ze removed the do not test disable testing on pull request label Feb 20, 2023
@rschu1ze rschu1ze changed the title (wip) replace{One,All}(): allow non-const pattern and replacement arguments replace{One,All}(): allow non-const pattern and replacement arguments Feb 20, 2023
@rschu1ze rschu1ze marked this pull request as ready for review February 20, 2023 19:11
@rschu1ze rschu1ze changed the title replace{One,All}(): allow non-const pattern and replacement arguments replace[Regexp]{One,All}(): allow non-const pattern and replacement arguments Feb 21, 2023
@rschu1ze

This comment was marked as outdated.

@CurtizJ CurtizJ self-assigned this Mar 31, 2023
@rschu1ze rschu1ze force-pushed the rs/nonconst-replace branch from 77e9a8b to 5238bbd Compare April 4, 2023 07:09
@rschu1ze rschu1ze force-pushed the rs/nonconst-replace branch from 5238bbd to a7c153f Compare April 4, 2023 08:26
@rschu1ze
Copy link
Copy Markdown
Member Author

rschu1ze commented Apr 5, 2023

00002_log_and_exception_messages_formatting is flaky --> #48441

@CurtizJ Approve? Thanks.

@CurtizJ CurtizJ merged commit 166a63c into master Apr 6, 2023
@CurtizJ CurtizJ deleted the rs/nonconst-replace branch April 6, 2023 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants