Merge pull request #61 from lphiri/master

Fix typo and error handling
projectatomic · Dec 2, 2016 · 0dd83ff · 0dd83ff
2 parents b43789f + 59ed725
commit 0dd83ff
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/linter.js b/lib/linter.js
@@ -24,7 +24,7 @@ var commandValidators = {
 
 function validateCommand(command, context, result) {
     if (command.error) {
-        helper. or(result, command.lineno, command.raw, command.error,null);
+        helper.addError(result, command.lineno, command.raw, command.error,null);
     }
     var rules = context.rules;
     //Check if command is a valid Dockerfile command

diff --git a/lib/parser.js b/lib/parser.js
@@ -347,7 +347,8 @@ function parseLine(line, lineno) {
     var commandParserFn = commandParsers[command.name];
     if (!commandParserFn) {
         // Invalid Dockerfile instruction, but allow it and move on.
-        // log.debug('Invalid Dockerfile command:', command.name);
+        console.log('Invalid Dockerfile command:', command.name);
+        command.error = 'Invalid Command';
         commandParserFn = parseString;
     }
     //LDMOD