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

repl: remove lastInputPreview conditional check by jenthone · Pull Request #46857 · nodejs/node · GitHub

/ node Public

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 commented Feb 27, 2023
edited
Loading

Copy link
Copy Markdown
Contributor

Fix #37585

Phenomenon: #37585 (comment)

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

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

cola119 commented Feb 28, 2023

Copy link
Copy Markdown
Member

cc @nodejs/repl @BridgeAR

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Feb 28, 2023

Copy link
Copy Markdown
Collaborator

BridgeAR 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

BridgeAR added author ready PRs that have at least one approval, no outstanding review comments, 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

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Collaborator

Landed in d0608c2

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>
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>
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
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 outstanding review comments, and a CI started. needs-ci PRs that need a full CI run. repl Issues 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


Back | FazBrowse Home | New Git URL