Merge pull request #720 from FooBarWidget/fix/linter-args

Pass valid filenames to RuboCop and Reek
rubyide · Sep 27, 2021 · 0b6cd1c · 0b6cd1c
2 parents da0d5d4 + 18d5a00
commit 0b6cd1c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/language-server-ruby/src/linters/Reek.ts b/packages/language-server-ruby/src/linters/Reek.ts
@@ -27,7 +27,7 @@ export default class Reek extends BaseLinter {
 
 	get args(): string[] {
 		const documentPath = URI.parse(this.document.uri);
-		return ['-f', 'json', '--stdin-filename', `'${documentPath.fsPath}'`];
+		return ['-f', 'json', '--stdin-filename', documentPath.fsPath];
 	}
 
 	protected processResults(data): Diagnostic[] {

diff --git a/packages/language-server-ruby/src/linters/RuboCop.ts b/packages/language-server-ruby/src/linters/RuboCop.ts
@@ -58,7 +58,7 @@ export default class RuboCop extends BaseLinter {
 
 	get args(): string[] {
 		const documentPath = URI.parse(this.document.uri);
-		let args = ['-s', `'${documentPath.fsPath}'`, '-f', 'json'];
+		let args = ['-s', documentPath.fsPath, '-f', 'json'];
 		if (this.lintConfig.rails) args.push('-R');
 		if (this.lintConfig.forceExclusion) args.push('--force-exclusion');
 		if (this.lintConfig.lint) args.push('-l');