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

errors: use `determineSpecificType` in more error messages by aduh95 · Pull Request #49580 · nodejs/node · GitHub

/ node Public

errors: use determineSpecificType in more error messages - #49580

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
aduh95:determineSpecificType
Sep 11, 2023
Merged

errors: use determineSpecificType in more error messages#49580
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
aduh95:determineSpecificType

Conversation

aduh95 commented Sep 9, 2023

Copy link
Copy Markdown
Contributor

Before it would have say something like Expected a … to be returned for the "…" from the function but got an instance of String., and with this PR it would say the more useful and more correct Expected a … to be returned for the "…" from the function but got type string ('the input string').

Fixes: #49576

nodejs-github-bot added errors Issues and PRs related to JavaScript errors originated in Node.js core. needs-ci PRs that need a full CI run. labels Sep 9, 2023
aduh95 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. labels Sep 10, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 10, 2023

Copy link
Copy Markdown
Collaborator

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 10, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 11, 2023
nodejs-github-bot merged commit d3e0331 into nodejs:main Sep 11, 2023

Copy link
Copy Markdown
Collaborator

Landed in d3e0331

ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
PR-URL: #49580
Fixes: #49576
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
This was referenced Sep 28, 2023
aduh95 deleted the determineSpecificType branch November 2, 2024 17:14
aduh95 added a commit that referenced this pull request Nov 2, 2024
PR-URL: #49580
Fixes: #49576
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matthew Aitken <maitken033380023@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. errors Issues and PRs related to JavaScript errors originated in Node.js core. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

loaders: super confusing "expected string but got instance of string"

4 participants


Back | FazBrowse Home | New Git URL