Skip to content

readline: replace _questionCancel with a symbol - #37094

Closed
cjihrig wants to merge 1 commit into
nodejs:masterfrom
cjihrig:symbol
Closed

readline: replace _questionCancel with a symbol#37094
cjihrig wants to merge 1 commit into
nodejs:masterfrom
cjihrig:symbol

Conversation

@cjihrig

Copy link
Copy Markdown
Contributor

This commit avoids exposing a new underscored property on
readline Interface instances.

This commit avoids exposing a new underscored property on
readline Interface instances.
@nodejs-github-botnodejs-github-bot added the readline Issues and PRs related to the built-in readline module. label Jan 27, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@LxxyxLxxyx added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 27, 2021
Lxxyx pushed a commit that referenced this pull request Jan 29, 2021
This commit avoids exposing a new underscored property on
readline Interface instances.
PR-URL: #37094
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Lxxyx

Copy link
Copy Markdown
Member

Landed in fec093b

@LxxyxLxxyx closed this Jan 29, 2021
@cjihrig
cjihrig deleted the symbol branch January 29, 2021 14:45
targos pushed a commit that referenced this pull request Feb 2, 2021
This commit avoids exposing a new underscored property on
readline Interface instances.
PR-URL: #37094
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targostargos mentioned this pull request Feb 2, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.readlineIssues and PRs related to the built-in readline module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@cjihrig@nodejs-github-bot@Lxxyx@jasnell@lpinca@targos