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: remove duplicate word in commit
  • Loading branch information
@not-an-aardvark
not-an-aardvark committed Apr 1, 2017
commit 55787017960dd93879bd78065c1dc0a3dac31f45
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 @@ -99,7 +99,7 @@ syntaxArgs.forEach(function(args) {
assert.strictEqual(c.status, 0, 'code == ' + c.status);
});

// should should throw if code piped from stdin with --check has bad syntax
// should throw if code piped from stdin with --check has bad syntax
// loop each possible option, `-c` or `--check`
syntaxArgs.forEach(function(args) {
const stdin = 'var foo bar;';
Expand Down