Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

esm: use correct error arguments #27763

Merged
merged 1 commit into from
May 20, 2019
Merged

esm: use correct error arguments #27763

merged 1 commit into from
May 20, 2019

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented May 18, 2019

ERR_UNKNOWN_FILE_EXTENSION expects a single argument. This
commit fixes the argument count.

Fixes: #27761

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented May 20, 2019

CI: https://ci.nodejs.org/job/node-test-pull-request/23230/

EDIT(cjihrig): CI was yellow.

ERR_UNKNOWN_FILE_EXTENSION expects a single argument. This
commit fixes the argument count.

PR-URL: nodejs#27763
Fixes: nodejs#27761
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@cjihrig cjihrig merged commit 873c372 into nodejs:master May 20, 2019
@cjihrig cjihrig deleted the esm-ext branch May 20, 2019 14:28
BridgeAR pushed a commit that referenced this pull request May 21, 2019
ERR_UNKNOWN_FILE_EXTENSION expects a single argument. This
commit fixes the argument count.

PR-URL: #27763
Fixes: #27761
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
@BridgeAR BridgeAR mentioned this pull request May 21, 2019
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ERR_UNKNOWN_FILE_EXTENSION is not thrown correctly
6 participants