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

doc: add history info for Permission Model #56707

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Jan 22, 2025

No description provided.

@aduh95 aduh95 added backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. backport-requested-v23.x PRs awaiting manual backport to the v23.x-staging branch. labels Jan 22, 2025
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 22, 2025
@aduh95 aduh95 added commit-queue Add this label to land a pull request using GitHub Actions. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Jan 22, 2025
@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Jan 22, 2025
Copy link
Contributor

Fast-track has been requested by @jasnell. Please 👍 to approve.

@jasnell
Copy link
Member

jasnell commented Jan 22, 2025

Adding fast-track simply because there's nothing here that should need to wait 48 hours to figure out.

@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 22, 2025
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/56707
✔  Done loading data for nodejs/node/pull/56707
----------------------------------- PR info ------------------------------------
Title      doc: add history info for Permission Model (#56707)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     aduh95:permission-model-history -> nodejs:main
Labels     doc, fast-track, author ready, backport-requested-v20.x, backport-requested-v22.x, backport-requested-v23.x
Commits    1
 - doc: add history info for Permission Model
Committers 1
 - Antoine du Hamel <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Wed, 22 Jan 2025 15:45:05 GMT
   ✔  Approvals: 2
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/56707#pullrequestreview-2567619096
   ✔  - Ruben Bridgewater (@BridgeAR) (TSC): https://github.com/nodejs/node/pull/56707#pullrequestreview-2567886558
   ℹ  This PR is being fast-tracked
   ✘  This PR needs to wait 44 more hours to land (or 0 hours if there is 1 more approval (👍) of the fast-track request from collaborators).
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/12915466398

@jasnell jasnell removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Jan 22, 2025
@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 22, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 22, 2025
@nodejs-github-bot nodejs-github-bot merged commit 309924f into nodejs:main Jan 22, 2025
50 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 309924f

@aduh95 aduh95 deleted the permission-model-history branch January 22, 2025 21:19
aduh95 added a commit that referenced this pull request Jan 23, 2025
PR-URL: #56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
aduh95 added a commit that referenced this pull request Jan 23, 2025
PR-URL: #56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
aduh95 added a commit to aduh95/node that referenced this pull request Jan 23, 2025
PR-URL: nodejs#56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
@aduh95 aduh95 removed backport-requested-v20.x PRs awaiting manual backport to the v20.x-staging branch. backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. backport-requested-v23.x PRs awaiting manual backport to the v23.x-staging branch. labels Jan 23, 2025
@aduh95 aduh95 added backport-open-v20.x Indicate that the PR has an open backport backported-to-v22.x PRs backported to the v22.x-staging branch. backported-to-v23.x PRs backported to the v23.x-staging branch. labels Jan 23, 2025
aduh95 added a commit to aduh95/node that referenced this pull request Jan 23, 2025
PR-URL: nodejs#56707
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
marco-ippolito pushed a commit that referenced this pull request Jan 23, 2025
PR-URL: #56707
Backport-PR-URL: #56724
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
marco-ippolito pushed a commit that referenced this pull request Jan 23, 2025
PR-URL: #56707
Backport-PR-URL: #56724
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. backport-open-v20.x Indicate that the PR has an open backport backported-to-v22.x PRs backported to the v22.x-staging branch. backported-to-v23.x PRs backported to the v23.x-staging branch. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants