Skip to content
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

[v22.x backport] doc: reserve ABI 130 for Electron 33 #54455

Closed

Conversation

clavin
Copy link
Contributor

@clavin clavin commented Aug 19, 2024

Manual backport of #54383 to v22.x.

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. v22.x v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch. labels Aug 19, 2024
@RedYetiDev
Copy link
Member

I'm not sure whether the branch to target should be v22.x or v22.x-staging

@clavin
Copy link
Contributor Author

clavin commented Aug 19, 2024

Found some docs about it and it looks like I need to potentially reformat my commit & indeed target v22.x-staging. Thanks for the note! I'll do that shortly.

@clavin clavin force-pushed the clavin/doc-electron-33-abi-for-v22 branch from 26ab808 to a574bd5 Compare August 19, 2024 16:02
@clavin clavin changed the base branch from v22.x to v22.x-staging August 19, 2024 16:03
@clavin clavin changed the title doc: reserve ABI 130 for Electron 33 [v22.x backport] doc: reserve ABI 130 for Electron 33 Aug 19, 2024
clavin added a commit to clavin/node that referenced this pull request Aug 19, 2024
Backport-PR-URL: nodejs#54455
PR-URL: nodejs#54383
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@clavin clavin force-pushed the clavin/doc-electron-33-abi-for-v22 branch from a574bd5 to 2013511 Compare August 19, 2024 16:04
@clavin
Copy link
Contributor Author

clavin commented Aug 19, 2024

👍 Alright, everything seems to be formatted properly now. I added the Backport-PR-URL: metadata to the backport commit here as well. (I assume it's supposed to point to this PR, but lmk if it should be something else!)

Backport-PR-URL: nodejs#54455
PR-URL: nodejs#54383
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@clavin clavin force-pushed the clavin/doc-electron-33-abi-for-v22 branch from 2013511 to a319641 Compare August 22, 2024 19:29
@clavin
Copy link
Contributor Author

clavin commented Aug 22, 2024

v22.x-staging got force-pushed to a new commit, so I rebased this PR to resolve the commit diffs.

RafaelGSS pushed a commit that referenced this pull request Aug 24, 2024
Backport-PR-URL: #54455
PR-URL: #54383
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@RafaelGSS
Copy link
Member

Landed in 83b5efe

@RafaelGSS RafaelGSS closed this Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. v22.x v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants