Skip to content

repl: remove lastInputPreview conditional check - #46857

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
jenthone:fix/prevent_identical_input
Mar 1, 2023
Merged

repl: remove lastInputPreview conditional check#46857
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
jenthone:fix/prevent_identical_input

Conversation

@jenthone

@jenthonejenthone commented Feb 27, 2023

Copy link
Copy Markdown
Contributor

Fix#37585

Phenomenon: #37585 (comment)

ChromeDevTools doesn’t show a preview identical to the input as well

ChromeDevTools

@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 Feb 27, 2023
@jenthone
jenthone marked this pull request as ready for review February 28, 2023 02:00
@cola119cola119 added the request-ci Add this label to start a Jenkins CI on a PR. label Feb 28, 2023
@cola119

Copy link
Copy Markdown
Member

cc @nodejs/repl @BridgeAR

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

Copy link
Copy Markdown
Collaborator

@BridgeARBridgeAR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@BridgeARBridgeAR added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. and removed request-ci Add this label to start a Jenkins CI on a PR. labels Feb 28, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@cola119cola119 added the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 1, 2023
@nodejs-github-bot
nodejs-github-bot merged commit d0608c2 into nodejs:mainMar 1, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in d0608c2

@jenthone
jenthone deleted the fix/prevent_identical_input branch March 1, 2023 15:46
targos pushed a commit that referenced this pull request Mar 13, 2023
PR-URL: #46857
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
targos pushed a commit that referenced this pull request Mar 14, 2023
PR-URL: #46857
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
@targostargos mentioned this pull request Mar 14, 2023
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
PR-URL: #46857
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
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.needs-ciPRs that need a full CI run.replIssues and PRs related to the REPL subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Strange repl eager-eval behavior

4 participants

@jenthone@cola119@nodejs-github-bot@BridgeAR