Merge pull request #1084 from palantir/config-cli-behavior

Treat all -c option CLI values as a path

src/tslint-cli.ts

@@ -18,6 +18,7 @@
 import * as fs from "fs";
 import * as glob from "glob";
 import * as optimist from "optimist";
+import * as path from "path";
 import * as Linter from "./tslint";
 import {
     CONFIG_FILENAME,
@@ -196,6 +197,7 @@ if (argv.c && !fs.existsSync(argv.c)) {
     console.error("Invalid option for configuration: " + argv.c);
     process.exit(1);
 }
+const possibleConfigAbsolutePath = argv.c != null ? path.resolve(argv.c) : null;
 
 const processFile = (file: string) => {
     if (!fs.existsSync(file)) {
@@ -204,7 +206,7 @@ const processFile = (file: string) => {
     }
 
     const contents = fs.readFileSync(file, "utf8");
-    const configuration = findConfiguration(argv.c, file);
+    const configuration = findConfiguration(possibleConfigAbsolutePath, file);
 
     const linter = new Linter(file, contents, {
         configuration,

test/check-bin.sh

@@ -66,6 +66,10 @@ expectOut $? 2 "tslint with with JSON pointing to two custom rules did not find
 ./bin/tslint -c ./test/config/tslint-custom-rules.json -r ./test/files/custom-rules --force src/tslint.ts
 expectOut $? 0 "tslint with with -r pointing to custom rules did not find lint failures"
 
+# make sure path to config without a preceding "./" works on the CLI
+./bin/tslint -c test/config/tslint-almost-empty.json src/tslint.ts
+expectOut $? 0 "-c relative path without ./ did not work"
+
 # make sure tslint --init generates a file
 cd ./bin
 if [ -f tslint.json ]; then

test/config/tslint-almost-empty.json

@@ -0,0 +1 @@
+{ "rules": {} }