Skip to content

lib: fix output message when repl is used with pm - #48438

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:adjust-repl-wording
Jun 17, 2023
Merged

lib: fix output message when repl is used with pm#48438
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:adjust-repl-wording

Conversation

@RafaelGSS

Copy link
Copy Markdown
Member

We use FileSytemWrite and FileSytemRead now.

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. repl Issues and PRs related to the REPL subsystem. labels Jun 12, 2023
@RafaelGSSRafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 13, 2023
@RafaelGSS

Copy link
Copy Markdown
MemberAuthor

cc: @nodejs/security-wg

@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 13, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@RafaelGSSRafaelGSS added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 17, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 17, 2023
@nodejs-github-bot
nodejs-github-bot merged commit ff14b24 into nodejs:mainJun 17, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in ff14b24

RafaelGSS added a commit that referenced this pull request Jul 3, 2023
PR-URL: #48438
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@RafaelGSSRafaelGSS mentioned this pull request Jul 3, 2023
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48438
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48438
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@ruyadorno

Copy link
Copy Markdown
Member

@RafaelGSS it looks like to me this message is not really in v18, please feel free to correct me if I'm wrong.

@RafaelGSSRafaelGSS added the permission Issues and PRs related to the Permission Model label Sep 19, 2023
@RafaelGSS

Copy link
Copy Markdown
MemberAuthor

Yes, that's correct @ruyadorno. Sorry about that. All the PRs targeting permission must not land in v18.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.permissionIssues and PRs related to the Permission ModelreplIssues and PRs related to the REPL subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RafaelGSS@nodejs-github-bot@ruyadorno@lpinca@tniessen