Skip to content

lib: change ERR_INVALID_ARG_VALUE error message - #35156

Closed
lundibundi wants to merge 2 commits into
nodejs:mainfrom
lundibundi:improve-arg-value-message
Closed

lib: change ERR_INVALID_ARG_VALUE error message#35156
lundibundi wants to merge 2 commits into
nodejs:mainfrom
lundibundi:improve-arg-value-message

Conversation

@lundibundi

Copy link
Copy Markdown
Member

Refs: #34682

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added ?
  • commit message follows commit guidelines

/cc @BridgeAR@jasnell@Trott

@nodejs-github-botnodejs-github-bot added the errors Issues and PRs related to JavaScript errors originated in Node.js core. label Sep 11, 2020
@lundibundilundibundi added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 11, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 11, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

code: 'ERR_INVALID_ARG_VALUE',
name: 'TypeError',
message: `The argument 'rrtype' is invalid. Received '${rrtype}'`
message: `The 'rrtype' is invalid. Received '${rrtype}' argument`

@TrottTrottSep 14, 2020

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.

Looks like argument ended up in the wrong place in this string?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Oh, good catch. Do we have a make command to run all tests?

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.

make test-all-suites

I would have used make test-all but it was already taken.

@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 16, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 16, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@lundibundi

Copy link
Copy Markdown
MemberAuthor

ping @BridgeAR@jasnell@Trott

@aduh95

Copy link
Copy Markdown
Contributor

This would need a rebase.

@aduh95aduh95 added the stalled Issues and PRs that are stalled. label Sep 19, 2023
@github-actions

Copy link
Copy Markdown
Contributor

This issue/PR was marked as stalled, it will be automatically closed in 30 days. If it should remain open, please leave a comment explaining why it should remain open.

@github-actions

Copy link
Copy Markdown
Contributor

Closing this because it has stalled. Feel free to reopen if this issue/PR is still relevant, or to ping the collaborator who labelled it stalled if you have any questions.

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

Labels

errorsIssues and PRs related to JavaScript errors originated in Node.js core.stalledIssues and PRs that are stalled.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@lundibundi@nodejs-github-bot@aduh95@Trott