FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: remove comma delimiter mention on permissions doc by RafaelGSS · Pull Request #58297 · nodejs/node · GitHub

/ node Public

doc: remove comma delimiter mention on permissions doc - #58297

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-doc-pm-comma
May 14, 2025
ConversationCommits (1)ChecksFiles changed
Merged

doc: remove comma delimiter mention on permissions doc#58297
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-doc-pm-comma

Conversation

RafaelGSS commented May 12, 2025

Copy link
Copy Markdown
Member

Fixes: #58287

nodejs-github-bot commented May 12, 2025

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

RafaelGSS added the permission Issues and PRs related to the Permission Model label May 12, 2025
nodejs-github-bot added the doc Issues and PRs related to the documentations. label May 12, 2025
aduh95 added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels May 12, 2025
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 14, 2025
nodejs-github-bot merged commit f75c960 into nodejs:main May 14, 2025

nodejs-github-bot commented May 14, 2025

Copy link
Copy Markdown
Collaborator

Landed in f75c960

targos pushed a commit that referenced this pull request May 16, 2025
doc: remove comma delimiter mention on permissions doc
3b9b010 
PR-URL: #58297
Fixes: #58287
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
github-actions Bot mentioned this pull request May 20, 2025
Merged
aduh95 pushed a commit that referenced this pull request Jun 10, 2025
doc: remove comma delimiter mention on permissions doc
39d9a61 
PR-URL: #58297
Fixes: #58287
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
github-actions Bot mentioned this pull request Jun 10, 2025
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jasnell jasnell approved these changes
lpinca lpinca approved these changes
UlisesGascon UlisesGascon approved these changes
geeksilva97 geeksilva97 approved these changes
marco-ippolito marco-ippolito approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. doc Issues and PRs related to the documentations. permission Issues and PRs related to the Permission Model

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Docs for --permission incorrectly state support for (,) delimiter

8 participants

Back | FazBrowse Home | New Git URL