Skip to content

Conversation

@RasmusWL
Copy link
Member

@RasmusWL RasmusWL commented Nov 9, 2021

As is already done in JS and Ruby.

As is already done in JS and Ruby.
@RasmusWL RasmusWL requested review from a team as code owners November 9, 2021 15:11
@RasmusWL RasmusWL added the no-change-note-required This PR does not need a change note label Nov 9, 2021
@RasmusWL
Copy link
Member Author

RasmusWL commented Nov 9, 2021

See discussion in #6129

Copy link
Contributor

@yoff yoff left a comment

Choose a reason for hiding this comment

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

LGTM

@yoff yoff merged commit 41b7922 into github:main Nov 29, 2021
@RasmusWL RasmusWL deleted the redos-cwe-1333 branch November 29, 2021 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C# no-change-note-required This PR does not need a change note Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants