Skip to content

Conversation

@anonrig
Copy link
Member

This is my first backport. If the time is right, I'd like to backport UTF8 encoding performance improvements, so that LTS users can access to the performance boosts.

I'm trying to backport the following changes to the v18 release line:

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch. labels Nov 27, 2022
@anonriganonrigforce-pushed the backport-utf8-to-v18.x branch from 4ad6874 to c1fd005CompareNovember 27, 2022 23:58
anonrigand others added 6 commits November 27, 2022 19:01
PR-URL: nodejs#45294 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Rich Trott <[email protected]>
PR-URL: nodejs#45363 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
PR-URL: nodejs#45363 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
PR-URL: nodejs#45363 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
PR-URL: nodejs#45388 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: Rich Trott <[email protected]>
Co-authored-by: Anna Henningsen <[email protected]> PR-URL: nodejs#45412 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]>
@anonriganonrigforce-pushed the backport-utf8-to-v18.x branch from c1fd005 to d86c37fCompareNovember 28, 2022 00:03
@targos
Copy link
Member

I appreciate the effort, but v18.x is Active LTS, so all changes should automatically be backported to it by releasers. You can add the lts-watch-v18.x label to make sure it happens.

@anonrig
Copy link
MemberAuthor

Thanks @targos!

@anonriganonrig closed this Nov 28, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.needs-ciPRs that need a full CI run.v18.xIssues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@anonrig@targos@nodejs-github-bot