Skip to content

cli: ensure --check flag works for piped-in code #11689

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.

Sign up for GitHub

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

Closed
not-an-aardvark wants to merge 7 commits into from
Closed

cli: ensure --check flag works for piped-in code #11689

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6e255ad
cli: ensure --check flag works for piped-in code
not-an-aardvark Mar 5, 2017
bc820d5
squash: ensure the filename is still used
not-an-aardvark Mar 6, 2017
24ad2bf
cli: throw when -c and -e are used simultaneously
not-an-aardvark Mar 30, 2017
913d121
squash: change exit code to 9 for invalid args
not-an-aardvark Mar 30, 2017
5578701
squash: remove duplicate word in commit
not-an-aardvark Mar 30, 2017
299155d
squash: make assertion error message more specific
not-an-aardvark Apr 1, 2017
fab87a9
squash: move flag validation to c++
not-an-aardvark Apr 1, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
squash: make assertion error message more specific
  • Loading branch information
@not-an-aardvark
not-an-aardvark committed Apr 1, 2017
commit 299155d554a51185b9870ec64c1348461b0ea41e
2 changes: 1 addition & 1 deletion test/parallel/test-cli-syntax.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,6 @@ syntaxArgs.forEach(function(args) {
'--check and --eval flags are mutually exclusive.\n'
);

assert.strictEqual(c.status, 9, 'code == ' + c.status);
assert.strictEqual(c.status, 9, 'code === ' + c.status);
});
});