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,permissions: adjust limitations and known issues section #51184

Merged

Conversation

RafaelGSS
Copy link
Member

This should avoid confusion against expected behaviour vs limitations.

@RafaelGSS RafaelGSS added the permission Issues and PRs related to the Permission Model label Dec 16, 2023
@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 Dec 16, 2023
@RafaelGSS RafaelGSS force-pushed the refactor-limitations-section-pm branch from 0a147b3 to 82a82c1 Compare December 16, 2023 16:23
@RafaelGSS RafaelGSS 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 Dec 18, 2023
@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 Dec 18, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/51184
✔  Done loading data for nodejs/node/pull/51184
----------------------------------- PR info ------------------------------------
Title      doc,permissions: adjust limitations and known issues section (#51184)
Author     Rafael Gonzaga  (@RafaelGSS)
Branch     RafaelGSS:refactor-limitations-section-pm -> nodejs:main
Labels     doc, author ready, permission
Commits    1
 - doc: fix limitations and known issues in pm
Committers 1
 - RafaelGSS 
PR-URL: https://github.com/nodejs/node/pull/51184
Reviewed-By: Yagiz Nizipli 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/51184
Reviewed-By: Yagiz Nizipli 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 16 Dec 2023 16:21:44 GMT
   ✔  Approvals: 1
   ✔  - Yagiz Nizipli (@anonrig) (TSC): https://github.com/nodejs/node/pull/51184#pullrequestreview-1785492638
   ✘  This PR needs to wait 114 more hours to land (or 0 hours if there is one more approval)
   ✔  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/7254037180

@RafaelGSS RafaelGSS removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Dec 19, 2023
Copy link
Contributor

@ShogunPanda ShogunPanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@RafaelGSS RafaelGSS added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 21, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 21, 2023
@nodejs-github-bot nodejs-github-bot merged commit b33a1d3 into nodejs:main Dec 21, 2023
25 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in b33a1d3

RafaelGSS added a commit that referenced this pull request Jan 2, 2024
PR-URL: #51184
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
@RafaelGSS RafaelGSS mentioned this pull request Jan 2, 2024
@aduh95 aduh95 mentioned this pull request Jan 7, 2024
richardlau pushed a commit that referenced this pull request Mar 25, 2024
PR-URL: #51184
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
@richardlau richardlau mentioned this pull request Mar 25, 2024
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. doc Issues and PRs related to the documentations. permission Issues and PRs related to the Permission Model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants