Skip to content

Conversation

@trop
Copy link
Contributor

@trop trop bot commented May 6, 2021

Backport of #29021

See that PR for details.

Notes: Improved performance of napi_threadsafe_function

@trop trop bot requested a review from a team as a code owner May 6, 2021 15:04
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label May 6, 2021
@trop trop bot mentioned this pull request May 6, 2021
5 tasks
@trop trop bot added 12-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels May 6, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label May 6, 2021
@indutny
Copy link
Contributor

indutny commented May 6, 2021

I believe the cherry-pick didn't work well in the case of this PR. There are extra lines in the patches/node/.patches that shouldn't be there cc @deepak1556 @zcbenz. Let me know if I can help by opening another PR with a proper backport.

@codebytere codebytere force-pushed the trop/12-x-y-bp-chore-cherry-pick-7abc7e45b2-from-node-1620313477600 branch from 37c5a19 to 481c4ec Compare May 6, 2021 19:11
@indutny-signal
Copy link
Contributor

@deepak1556 @zcbenz sorry for an extra ping, but it'd mean a lot for me if this could go into the next 12-x-y release. Would appreciate a review. Thanks!

@MarshallOfSound MarshallOfSound merged commit 47431a5 into 12-x-y May 7, 2021
@release-clerk
Copy link

release-clerk bot commented May 7, 2021

Release Notes Persisted

Improved performance of napi_threadsafe_function

@MarshallOfSound MarshallOfSound deleted the trop/12-x-y-bp-chore-cherry-pick-7abc7e45b2-from-node-1620313477600 branch May 7, 2021 07:20
@indutny-signal
Copy link
Contributor

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

12-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants