diff options
author | Markus Doits <markus.doits@stellenticket.de> | 2016-09-11 14:07:42 +0200 |
---|---|---|
committer | Markus Doits <markus.doits@stellenticket.de> | 2016-09-11 14:07:42 +0200 |
commit | 3403f9d12247884c18ffe7a1636fe12c3fb0f0da (patch) | |
tree | a343b6b25b22553489e95c3ffc93e5b9502f03b9 /lib | |
parent | 7c0daa97e07ca409d1ee9f394adb4967196c6ca1 (diff) | |
download | pronto-hlint-3403f9d12247884c18ffe7a1636fe12c3fb0f0da.tar.gz pronto-hlint-3403f9d12247884c18ffe7a1636fe12c3fb0f0da.tar.zst pronto-hlint-3403f9d12247884c18ffe7a1636fe12c3fb0f0da.zip |
add class variables to prepare setting external config
Diffstat (limited to 'lib')
-rw-r--r-- | lib/pronto/eslint_npm.rb | 25 |
1 files changed, 19 insertions, 6 deletions
diff --git a/lib/pronto/eslint_npm.rb b/lib/pronto/eslint_npm.rb index 4f32e5b..6bbef09 100644 --- a/lib/pronto/eslint_npm.rb +++ b/lib/pronto/eslint_npm.rb | |||
@@ -3,15 +3,30 @@ require 'shellwords' | |||
3 | 3 | ||
4 | module Pronto | 4 | module Pronto |
5 | class ESLintNpm < Runner | 5 | class ESLintNpm < Runner |
6 | class << self | ||
7 | attr_writer :eslint_executable, :files_to_lint | ||
8 | |||
9 | def eslint_executable | ||
10 | @eslint_executable || 'eslint'.freeze | ||
11 | end | ||
12 | |||
13 | def files_to_lint | ||
14 | @files_to_lint || /(\.js|\.es6)$/ | ||
15 | end | ||
16 | end | ||
17 | |||
6 | def run | 18 | def run |
7 | return [] unless @patches | 19 | return [] unless @patches |
8 | 20 | ||
9 | @patches.select { |patch| patch.additions > 0 } | 21 | @patches |
22 | .select { |patch| patch.additions > 0 } | ||
10 | .select { |patch| js_file?(patch.new_file_full_path) } | 23 | .select { |patch| js_file?(patch.new_file_full_path) } |
11 | .map { |patch| inspect(patch) } | 24 | .map { |patch| inspect(patch) } |
12 | .flatten.compact | 25 | .flatten.compact |
13 | end | 26 | end |
14 | 27 | ||
28 | private | ||
29 | |||
15 | def inspect(patch) | 30 | def inspect(patch) |
16 | @_repo_path ||= @patches.first.repo.path | 31 | @_repo_path ||= @patches.first.repo.path |
17 | 32 | ||
@@ -25,8 +40,6 @@ module Pronto | |||
25 | end | 40 | end |
26 | end | 41 | end |
27 | 42 | ||
28 | private | ||
29 | |||
30 | def new_message(offence, line) | 43 | def new_message(offence, line) |
31 | path = line.patch.delta.new_file[:path] | 44 | path = line.patch.delta.new_file[:path] |
32 | level = :warning | 45 | level = :warning |
@@ -35,18 +48,18 @@ module Pronto | |||
35 | end | 48 | end |
36 | 49 | ||
37 | def js_file?(path) | 50 | def js_file?(path) |
38 | %w(.js .es6 .js.es6).include?(File.extname(path)) | 51 | self.class.files_to_lint =~ path.to_s |
39 | end | 52 | end |
40 | 53 | ||
41 | def run_eslint(patch) | 54 | def run_eslint(patch) |
42 | Dir.chdir(@_repo_path) do | 55 | Dir.chdir(@_repo_path) do |
56 | escaped_file_path = Shellwords.escape(patch.new_file_full_path.to_s) | ||
43 | JSON.parse( | 57 | JSON.parse( |
44 | `eslint #{Shellwords.escape(patch.new_file_full_path.to_s)} -f json` | 58 | `#{self.class.eslint_executable} #{escaped_file_path} -f json` |
45 | ) | 59 | ) |
46 | end | 60 | end |
47 | end | 61 | end |
48 | 62 | ||
49 | # rubocop:disable Metrics/LineLength | ||
50 | def clean_up_eslint_output(output) | 63 | def clean_up_eslint_output(output) |
51 | # 1. Filter out offences without a warning or error | 64 | # 1. Filter out offences without a warning or error |
52 | # 2. Get the messages for that file | 65 | # 2. Get the messages for that file |