-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
doc: mention native addons are restricted in pm #51185
doc: mention native addons are restricted in pm #51185
Conversation
Review requested:
|
Co-authored-by: Tobias Nießen <tniessen@tnie.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Commit Queue failed- Loading data for nodejs/node/pull/51185 ✔ Done loading data for nodejs/node/pull/51185 ----------------------------------- PR info ------------------------------------ Title doc: mention native addons are restricted in pm (#51185) Author Rafael Gonzaga (@RafaelGSS) Branch RafaelGSS:mention-addons-are-restricted -> nodejs:main Labels doc, author ready, permission Commits 2 - doc: mention native addons are restricted in pm - fixup! typo Committers 2 - RafaelGSS - GitHub PR-URL: https://github.com/nodejs/node/pull/51185 Reviewed-By: Marco Ippolito Reviewed-By: Michael Dawson ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/51185 Reviewed-By: Marco Ippolito Reviewed-By: Michael Dawson -------------------------------------------------------------------------------- ℹ This PR was created on Sat, 16 Dec 2023 16:26:17 GMT ✔ Approvals: 2 ✔ - Marco Ippolito (@marco-ippolito): https://github.com/nodejs/node/pull/51185#pullrequestreview-1785242443 ✔ - Michael Dawson (@mhdawson) (TSC): https://github.com/nodejs/node/pull/51185#pullrequestreview-1787536430 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 51185 From https://github.com/nodejs/node * branch refs/pull/51185/merge -> FETCH_HEAD ✔ Fetched commits as 135948d584ab..2483f9125ae2 -------------------------------------------------------------------------------- [main 354bade593] doc: mention native addons are restricted in pm Author: RafaelGSS Date: Sat Dec 16 13:25:34 2023 -0300 1 file changed, 1 insertion(+), 1 deletion(-) [main 7992aad966] fixup! typo Author: Rafael Gonzaga Date: Mon Dec 18 10:43:41 2023 -0300 1 file changed, 1 insertion(+), 1 deletion(-) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)https://github.com/nodejs/node/actions/runs/7254037180 |
Landed in 27d953d |
PR-URL: #51185 Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>
PR-URL: #51185 Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>
Clarifying native addons are restricted by default when using the permission model.