danger-rubocop 0.7.0 → 0.9.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.circleci/config.yml +30 -0
- data/Gemfile +1 -1
- data/Gemfile.lock +33 -24
- data/README.md +8 -1
- data/danger-rubocop.gemspec +2 -2
- data/lib/danger_plugin.rb +13 -8
- data/lib/version.rb +1 -1
- data/spec/danger_plugin_spec.rb +51 -12
- metadata +10 -11
- data/circle.yml +0 -3
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: a3abbdb817a34c89263a3e43389e0ad1c241eceb220da6ff60d4171dbd82724b
|
4
|
+
data.tar.gz: 7790b03ff4cffe993a9e21ada5831169e55af7811bb06b2b6dc800ee0eab72c4
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: a6532f9058d5f6f5db6fd27c1197b84966ef5e786fd9213c24974d618477c5f7eabfadb8f072585a3e6c413f08d2f67aa94610486c599e91fc8fc528e2a2d36e
|
7
|
+
data.tar.gz: c678e3db688d226c8d25c43bc895f046519b14751bb0b3b670b124ca8a90f884d9695b2fb936c98cd38ad7899127b4539d8eb3b9e4a39d622a23f758f9e27c34
|
@@ -0,0 +1,30 @@
|
|
1
|
+
version: 2
|
2
|
+
jobs:
|
3
|
+
build:
|
4
|
+
docker:
|
5
|
+
- image: circleci/ruby:2.6
|
6
|
+
|
7
|
+
working_directory: ~/repo
|
8
|
+
|
9
|
+
steps:
|
10
|
+
- checkout
|
11
|
+
|
12
|
+
- restore_cache:
|
13
|
+
keys:
|
14
|
+
- v1-dependencies-{{ checksum "Gemfile.lock" }}
|
15
|
+
# fallback to using the latest cache if no exact match is found
|
16
|
+
- v1-dependencies-
|
17
|
+
|
18
|
+
- run:
|
19
|
+
name: install dependencies
|
20
|
+
command: |
|
21
|
+
bundle install --jobs=4 --retry=3 --path vendor/bundle
|
22
|
+
|
23
|
+
- save_cache:
|
24
|
+
paths:
|
25
|
+
- ./vendor/bundle
|
26
|
+
key: v1-dependencies-{{ checksum "Gemfile.lock" }}
|
27
|
+
|
28
|
+
- run:
|
29
|
+
name: run tests
|
30
|
+
command: bundle exec rake spec
|
data/Gemfile
CHANGED
data/Gemfile.lock
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
PATH
|
2
2
|
remote: .
|
3
3
|
specs:
|
4
|
-
danger-rubocop (0.
|
4
|
+
danger-rubocop (0.8.0)
|
5
5
|
danger
|
6
6
|
rubocop
|
7
7
|
|
@@ -10,7 +10,7 @@ GEM
|
|
10
10
|
specs:
|
11
11
|
addressable (2.7.0)
|
12
12
|
public_suffix (>= 2.0.2, < 5.0)
|
13
|
-
ast (2.4.
|
13
|
+
ast (2.4.1)
|
14
14
|
bacon (1.2.0)
|
15
15
|
claide (1.0.3)
|
16
16
|
claide-plugins (0.9.2)
|
@@ -21,27 +21,28 @@ GEM
|
|
21
21
|
colored2 (3.1.2)
|
22
22
|
cork (0.3.0)
|
23
23
|
colored2 (~> 3.1)
|
24
|
-
danger (
|
24
|
+
danger (8.0.1)
|
25
25
|
claide (~> 1.0)
|
26
26
|
claide-plugins (>= 0.9.2)
|
27
27
|
colored2 (~> 3.1)
|
28
28
|
cork (~> 0.1)
|
29
|
-
faraday (
|
29
|
+
faraday (>= 0.9.0, < 2.0)
|
30
30
|
faraday-http-cache (~> 2.0)
|
31
|
-
git (~> 1.
|
31
|
+
git (~> 1.7)
|
32
32
|
kramdown (~> 2.0)
|
33
33
|
kramdown-parser-gfm (~> 1.0)
|
34
34
|
no_proxy_fix
|
35
35
|
octokit (~> 4.7)
|
36
36
|
terminal-table (~> 1)
|
37
37
|
diff-lcs (1.2.5)
|
38
|
-
faraday (0.
|
38
|
+
faraday (1.0.1)
|
39
39
|
multipart-post (>= 1.2, < 3)
|
40
|
-
faraday-http-cache (2.
|
41
|
-
faraday (
|
40
|
+
faraday-http-cache (2.2.0)
|
41
|
+
faraday (>= 0.8)
|
42
42
|
ffi (1.9.10)
|
43
43
|
formatador (0.2.5)
|
44
|
-
git (1.
|
44
|
+
git (1.7.0)
|
45
|
+
rchardet (~> 1.8)
|
45
46
|
guard (2.14.0)
|
46
47
|
formatador (>= 0.2.4)
|
47
48
|
listen (>= 2.7, < 4.0)
|
@@ -56,8 +57,8 @@ GEM
|
|
56
57
|
guard (~> 2.1)
|
57
58
|
guard-compat (~> 1.1)
|
58
59
|
rspec (>= 2.99.0, < 4.0)
|
59
|
-
|
60
|
-
|
60
|
+
kramdown (2.3.0)
|
61
|
+
rexml
|
61
62
|
kramdown-parser-gfm (1.1.0)
|
62
63
|
kramdown (~> 2.0)
|
63
64
|
listen (3.0.7)
|
@@ -77,11 +78,12 @@ GEM
|
|
77
78
|
notiffany (0.1.0)
|
78
79
|
nenv (~> 0.1)
|
79
80
|
shellany (~> 0.0)
|
80
|
-
octokit (4.
|
81
|
+
octokit (4.18.0)
|
82
|
+
faraday (>= 0.9)
|
81
83
|
sawyer (~> 0.8.0, >= 0.5.3)
|
82
84
|
open4 (1.3.4)
|
83
|
-
parallel (1.
|
84
|
-
parser (2.
|
85
|
+
parallel (1.19.1)
|
86
|
+
parser (2.7.1.3)
|
85
87
|
ast (~> 2.4.0)
|
86
88
|
prettybacon (0.0.2)
|
87
89
|
bacon (~> 1.2)
|
@@ -89,12 +91,15 @@ GEM
|
|
89
91
|
coderay (~> 1.1.0)
|
90
92
|
method_source (~> 0.8.1)
|
91
93
|
slop (~> 3.4)
|
92
|
-
public_suffix (4.0.
|
94
|
+
public_suffix (4.0.5)
|
93
95
|
rainbow (3.0.0)
|
94
|
-
rake (
|
96
|
+
rake (13.0.1)
|
95
97
|
rb-fsevent (0.9.7)
|
96
98
|
rb-inotify (0.9.7)
|
97
99
|
ffi (>= 0.5.0)
|
100
|
+
rchardet (1.8.0)
|
101
|
+
regexp_parser (1.7.1)
|
102
|
+
rexml (3.2.4)
|
98
103
|
rspec (3.4.0)
|
99
104
|
rspec-core (~> 3.4.0)
|
100
105
|
rspec-expectations (~> 3.4.0)
|
@@ -108,13 +113,17 @@ GEM
|
|
108
113
|
diff-lcs (>= 1.2.0, < 2.0)
|
109
114
|
rspec-support (~> 3.4.0)
|
110
115
|
rspec-support (3.4.1)
|
111
|
-
rubocop (0.
|
112
|
-
jaro_winkler (~> 1.5.1)
|
116
|
+
rubocop (0.85.1)
|
113
117
|
parallel (~> 1.10)
|
114
|
-
parser (>= 2.
|
118
|
+
parser (>= 2.7.0.1)
|
115
119
|
rainbow (>= 2.2.2, < 4.0)
|
120
|
+
regexp_parser (>= 1.7)
|
121
|
+
rexml
|
122
|
+
rubocop-ast (>= 0.0.3)
|
116
123
|
ruby-progressbar (~> 1.7)
|
117
|
-
unicode-display_width (>= 1.4.0, <
|
124
|
+
unicode-display_width (>= 1.4.0, < 2.0)
|
125
|
+
rubocop-ast (0.0.3)
|
126
|
+
parser (>= 2.7.0.1)
|
118
127
|
ruby-progressbar (1.10.1)
|
119
128
|
sawyer (0.8.2)
|
120
129
|
addressable (>= 2.3.5)
|
@@ -124,8 +133,8 @@ GEM
|
|
124
133
|
terminal-table (1.8.0)
|
125
134
|
unicode-display_width (~> 1.1, >= 1.1.1)
|
126
135
|
thor (0.19.1)
|
127
|
-
unicode-display_width (1.
|
128
|
-
yard (0.9.
|
136
|
+
unicode-display_width (1.7.0)
|
137
|
+
yard (0.9.20)
|
129
138
|
|
130
139
|
PLATFORMS
|
131
140
|
ruby
|
@@ -141,12 +150,12 @@ DEPENDENCIES
|
|
141
150
|
mocha-on-bacon
|
142
151
|
prettybacon
|
143
152
|
pry
|
144
|
-
rake (~>
|
153
|
+
rake (~> 13.0)
|
145
154
|
rspec (~> 3.4)
|
146
155
|
yard
|
147
156
|
|
148
157
|
RUBY VERSION
|
149
|
-
ruby 2.
|
158
|
+
ruby 2.6.0p0
|
150
159
|
|
151
160
|
BUNDLED WITH
|
152
161
|
1.17.3
|
data/README.md
CHANGED
@@ -30,6 +30,12 @@ public_files = (git.modified_files + git.added_files).select { |path| path.inclu
|
|
30
30
|
rubocop.lint public_files
|
31
31
|
```
|
32
32
|
|
33
|
+
> Submit comments only for changed lines
|
34
|
+
|
35
|
+
```ruby
|
36
|
+
github.dismiss_out_of_range_messages
|
37
|
+
rubocop.lint inline_comment: true
|
38
|
+
```
|
33
39
|
|
34
40
|
#### Methods
|
35
41
|
|
@@ -37,7 +43,7 @@ rubocop.lint public_files
|
|
37
43
|
|
38
44
|
Runs Ruby files through Rubocop. Generates a `markdown` list of warnings.
|
39
45
|
|
40
|
-
This method accepts configuration hash.
|
46
|
+
This method accepts a configuration hash.
|
41
47
|
The following keys are supported:
|
42
48
|
|
43
49
|
* `files`: array of file names or glob patterns to determine files to lint
|
@@ -50,6 +56,7 @@ The following keys are supported:
|
|
50
56
|
* `config`: path to the `.rubocop.yml` file.
|
51
57
|
* `only_report_new_offenses`: pass `true` to only report offenses that are in current user's scope.
|
52
58
|
Note that this won't mark offenses for _Metrics/XXXLength_ if you add lines to an already existing scope.
|
59
|
+
* `include_cop_names`: Prepends cop names to the output messages. Example: "Layout/EmptyLinesAroundBlockBody: Extra empty line detected at block body end."
|
53
60
|
|
54
61
|
|
55
62
|
Passing `files` as only argument is also supported for backward compatibility.
|
data/danger-rubocop.gemspec
CHANGED
@@ -18,11 +18,11 @@ Gem::Specification.new do |spec|
|
|
18
18
|
spec.require_paths = ['lib']
|
19
19
|
|
20
20
|
spec.add_dependency 'danger'
|
21
|
-
spec.add_dependency 'rubocop'
|
21
|
+
spec.add_dependency 'rubocop', '~> 0.83'
|
22
22
|
|
23
23
|
# General ruby development
|
24
24
|
spec.add_development_dependency 'bundler', '~> 1.3'
|
25
|
-
spec.add_development_dependency 'rake', '~>
|
25
|
+
spec.add_development_dependency 'rake', '~> 13.0'
|
26
26
|
|
27
27
|
# For validating the plugin lints
|
28
28
|
spec.add_development_dependency 'yard'
|
data/lib/danger_plugin.rb
CHANGED
@@ -35,6 +35,7 @@ module Danger
|
|
35
35
|
only_report_new_offenses = config[:only_report_new_offenses] || false
|
36
36
|
inline_comment = config[:inline_comment] || false
|
37
37
|
fail_on_inline_comment = config[:fail_on_inline_comment] || false
|
38
|
+
include_cop_names = config[:include_cop_names] || false
|
38
39
|
|
39
40
|
files_to_lint = fetch_files_to_lint(files)
|
40
41
|
files_to_report = rubocop(files_to_lint, force_exclusion, only_report_new_offenses, config_path: config_path)
|
@@ -43,16 +44,16 @@ module Danger
|
|
43
44
|
return report_failures files_to_report if report_danger
|
44
45
|
|
45
46
|
if inline_comment
|
46
|
-
add_violation_for_each_line(files_to_report, fail_on_inline_comment)
|
47
|
+
add_violation_for_each_line(files_to_report, fail_on_inline_comment, include_cop_names: include_cop_names)
|
47
48
|
else
|
48
|
-
markdown offenses_message(files_to_report)
|
49
|
+
markdown offenses_message(files_to_report, include_cop_names: include_cop_names)
|
49
50
|
end
|
50
51
|
end
|
51
52
|
|
52
53
|
private
|
53
54
|
|
54
55
|
def rubocop(files_to_lint, force_exclusion, only_report_new_offenses, config_path: nil)
|
55
|
-
base_command = ['rubocop', '-f', 'json']
|
56
|
+
base_command = ['rubocop', '-f', 'json', '--only-recognized-file-types']
|
56
57
|
base_command.concat(['--force-exclusion']) if force_exclusion
|
57
58
|
base_command.concat(['--config', config_path.shellescape]) unless config_path.nil?
|
58
59
|
|
@@ -86,14 +87,14 @@ module Danger
|
|
86
87
|
# Get start from diff.
|
87
88
|
lineno = first_line.match(/\+(\d+),(\d+)/).captures.first.to_i
|
88
89
|
diff.each_with_object([]) do |current_line, added_lines|
|
89
|
-
lineno += 1 unless current_line.start_with?('-')
|
90
90
|
added_lines << lineno if current_line.start_with?('+')
|
91
|
+
lineno += 1 unless current_line.start_with?('-')
|
91
92
|
added_lines
|
92
93
|
end
|
93
94
|
end
|
94
95
|
end
|
95
96
|
|
96
|
-
def offenses_message(offending_files)
|
97
|
+
def offenses_message(offending_files, include_cop_names: false)
|
97
98
|
require 'terminal-table'
|
98
99
|
|
99
100
|
message = "### Rubocop violations\n\n"
|
@@ -102,7 +103,9 @@ module Danger
|
|
102
103
|
style: { border_i: '|' },
|
103
104
|
rows: offending_files.flat_map do |file|
|
104
105
|
file['offenses'].map do |offense|
|
105
|
-
|
106
|
+
offense_message = offense['message']
|
107
|
+
offense_message = offense['cop_name'] + ': ' + offense_message if include_cop_names
|
108
|
+
[file['path'], offense['location']['line'], offense_message]
|
106
109
|
end
|
107
110
|
end
|
108
111
|
).to_s
|
@@ -117,11 +120,13 @@ module Danger
|
|
117
120
|
end
|
118
121
|
end
|
119
122
|
|
120
|
-
def add_violation_for_each_line(offending_files, fail_on_inline_comment)
|
123
|
+
def add_violation_for_each_line(offending_files, fail_on_inline_comment, include_cop_names: false)
|
121
124
|
offending_files.flat_map do |file|
|
122
125
|
file['offenses'].map do |offense|
|
126
|
+
offense_message = offense['message']
|
127
|
+
offense_message = offense['cop_name'] + ': ' + offense_message if include_cop_names
|
123
128
|
arguments = [
|
124
|
-
|
129
|
+
offense_message,
|
125
130
|
{
|
126
131
|
file: file['path'],
|
127
132
|
line: offense['location']['line']
|
data/lib/version.rb
CHANGED
data/spec/danger_plugin_spec.rb
CHANGED
@@ -20,7 +20,7 @@ module Danger
|
|
20
20
|
'offenses' => [
|
21
21
|
{
|
22
22
|
'message' => 'No.',
|
23
|
-
'location' => { 'line' =>
|
23
|
+
'location' => { 'line' => 41 }
|
24
24
|
}
|
25
25
|
]
|
26
26
|
}
|
@@ -79,7 +79,7 @@ module Danger
|
|
79
79
|
'offenses' => [
|
80
80
|
{
|
81
81
|
'message' => 'No.',
|
82
|
-
'location' => { 'line' =>
|
82
|
+
'location' => { 'line' => 41 }
|
83
83
|
}
|
84
84
|
]
|
85
85
|
}
|
@@ -88,6 +88,27 @@ module Danger
|
|
88
88
|
end
|
89
89
|
end
|
90
90
|
|
91
|
+
describe :added_lines do
|
92
|
+
before do
|
93
|
+
allow(@rubocop.git).to receive(:diff_for_file).with('SAMPLE') do
|
94
|
+
instance_double('Git::Diff::DiffFile', patch: <<~DIFF)
|
95
|
+
diff --git a/SAMPLE b/SAMPLE
|
96
|
+
new file mode 100644
|
97
|
+
index 0000000..7bba8c8
|
98
|
+
--- /dev/null
|
99
|
+
+++ b/SAMPLE
|
100
|
+
@@ -0,0 +1,2 @@
|
101
|
+
+line 1
|
102
|
+
+line 2
|
103
|
+
DIFF
|
104
|
+
end
|
105
|
+
end
|
106
|
+
|
107
|
+
it 'handles git diff' do
|
108
|
+
expect(@rubocop.send(:added_lines, 'SAMPLE')).to eq([1, 2])
|
109
|
+
end
|
110
|
+
end
|
111
|
+
|
91
112
|
describe :lint_files do
|
92
113
|
let(:response_ruby_file) do
|
93
114
|
{
|
@@ -96,6 +117,7 @@ module Danger
|
|
96
117
|
'path' => 'spec/fixtures/ruby_file.rb',
|
97
118
|
'offenses' => [
|
98
119
|
{
|
120
|
+
'cop_name' => 'Syntax/WhetherYouShouldDoThat',
|
99
121
|
'message' => "Don't do that!",
|
100
122
|
'location' => { 'line' => 13 }
|
101
123
|
}
|
@@ -112,6 +134,7 @@ module Danger
|
|
112
134
|
'path' => 'spec/fixtures/another_ruby_file.rb',
|
113
135
|
'offenses' => [
|
114
136
|
{
|
137
|
+
'cop_name' => 'Syntax/WhetherYouShouldDoThat',
|
115
138
|
'message' => "Don't do that!",
|
116
139
|
'location' => { 'line' => 23 }
|
117
140
|
}
|
@@ -123,7 +146,7 @@ module Danger
|
|
123
146
|
|
124
147
|
it 'handles a rubocop report for specified files' do
|
125
148
|
allow(@rubocop).to receive(:`)
|
126
|
-
.with('bundle exec rubocop -f json --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
|
149
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
|
127
150
|
.and_return(response_ruby_file)
|
128
151
|
|
129
152
|
# Do it
|
@@ -137,9 +160,25 @@ module Danger
|
|
137
160
|
expect(output).to include("spec/fixtures/ruby_file.rb | 13 | Don't do that!")
|
138
161
|
end
|
139
162
|
|
163
|
+
it 'includes cop names when include_cop_names is set' do
|
164
|
+
allow(@rubocop).to receive(:`)
|
165
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
|
166
|
+
.and_return(response_ruby_file)
|
167
|
+
|
168
|
+
# Do it
|
169
|
+
@rubocop.lint(files: 'spec/fixtures/ruby*.rb', config: 'path/to/rubocop.yml', include_cop_names: true)
|
170
|
+
|
171
|
+
output = @rubocop.status_report[:markdowns].first.message
|
172
|
+
|
173
|
+
# A title
|
174
|
+
expect(output).to include('Rubocop violations')
|
175
|
+
# A warning
|
176
|
+
expect(output).to include("spec/fixtures/ruby_file.rb | 13 | Syntax/WhetherYouShouldDoThat: Don't do that!")
|
177
|
+
end
|
178
|
+
|
140
179
|
it 'handles a rubocop report for specified files (legacy)' do
|
141
180
|
allow(@rubocop).to receive(:`)
|
142
|
-
.with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
|
181
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
|
143
182
|
.and_return(response_ruby_file)
|
144
183
|
|
145
184
|
# Do it
|
@@ -155,7 +194,7 @@ module Danger
|
|
155
194
|
|
156
195
|
it 'appends --force-exclusion argument when force_exclusion is set' do
|
157
196
|
allow(@rubocop).to receive(:`)
|
158
|
-
.with('bundle exec rubocop -f json --force-exclusion spec/fixtures/ruby_file.rb')
|
197
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types --force-exclusion spec/fixtures/ruby_file.rb')
|
159
198
|
.and_return(response_ruby_file)
|
160
199
|
|
161
200
|
# Do it
|
@@ -175,7 +214,7 @@ module Danger
|
|
175
214
|
.and_return(["spec/fixtures/another_ruby_file.rb"])
|
176
215
|
|
177
216
|
allow(@rubocop).to receive(:`)
|
178
|
-
.with('bundle exec rubocop -f json spec/fixtures/another_ruby_file.rb')
|
217
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/another_ruby_file.rb')
|
179
218
|
.and_return(response_another_ruby_file)
|
180
219
|
|
181
220
|
@rubocop.lint
|
@@ -191,7 +230,7 @@ module Danger
|
|
191
230
|
.and_return(['spec/fixtures/ruby_file.rb'])
|
192
231
|
allow(@rubocop.git).to receive(:added_files).and_return([])
|
193
232
|
allow(@rubocop).to receive(:`)
|
194
|
-
.with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
|
233
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
|
195
234
|
.and_return(response_ruby_file)
|
196
235
|
|
197
236
|
@rubocop.lint
|
@@ -213,13 +252,13 @@ EOS
|
|
213
252
|
.and_return(['spec/fixtures/ruby_file.rb'])
|
214
253
|
allow(@rubocop.git).to receive(:added_files).and_return([])
|
215
254
|
allow(@rubocop).to receive(:`)
|
216
|
-
.with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
|
255
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
|
217
256
|
.and_return(response_ruby_file)
|
218
257
|
|
219
258
|
@rubocop.lint(inline_comment: true)
|
220
259
|
|
221
260
|
expect(@rubocop.violation_report[:warnings].first.to_s)
|
222
|
-
.to eq("Violation Don't do that! { sticky: false, file: spec/fixtures/ruby_file.rb, line: 13 }")
|
261
|
+
.to eq("Violation Don't do that! { sticky: false, file: spec/fixtures/ruby_file.rb, line: 13, type: warning }")
|
223
262
|
end
|
224
263
|
end
|
225
264
|
|
@@ -229,13 +268,13 @@ EOS
|
|
229
268
|
.and_return(['spec/fixtures/ruby_file.rb'])
|
230
269
|
allow(@rubocop.git).to receive(:added_files).and_return([])
|
231
270
|
allow(@rubocop).to receive(:`)
|
232
|
-
.with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
|
271
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
|
233
272
|
.and_return(response_ruby_file)
|
234
273
|
|
235
274
|
@rubocop.lint(fail_on_inline_comment: true, inline_comment: true)
|
236
275
|
|
237
276
|
expect(@rubocop.violation_report[:errors].first.to_s)
|
238
|
-
.to eq("Violation Don't do that! { sticky: false, file: spec/fixtures/ruby_file.rb, line: 13 }")
|
277
|
+
.to eq("Violation Don't do that! { sticky: false, file: spec/fixtures/ruby_file.rb, line: 13, type: error }")
|
239
278
|
end
|
240
279
|
end
|
241
280
|
end
|
@@ -262,7 +301,7 @@ EOS
|
|
262
301
|
.and_return(['spec/fixtures/ruby_file.rb'])
|
263
302
|
allow(@rubocop.git).to receive(:added_files).and_return([])
|
264
303
|
allow(@rubocop).to receive(:`)
|
265
|
-
.with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
|
304
|
+
.with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
|
266
305
|
.and_return(response_ruby_file)
|
267
306
|
|
268
307
|
expect(@rubocop).to receive(:fail).with(fail_msg)
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: danger-rubocop
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.
|
4
|
+
version: 0.9.0
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Ash Furrow
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date:
|
11
|
+
date: 2020-08-12 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: danger
|
@@ -28,16 +28,16 @@ dependencies:
|
|
28
28
|
name: rubocop
|
29
29
|
requirement: !ruby/object:Gem::Requirement
|
30
30
|
requirements:
|
31
|
-
- - "
|
31
|
+
- - "~>"
|
32
32
|
- !ruby/object:Gem::Version
|
33
|
-
version: '0'
|
33
|
+
version: '0.83'
|
34
34
|
type: :runtime
|
35
35
|
prerelease: false
|
36
36
|
version_requirements: !ruby/object:Gem::Requirement
|
37
37
|
requirements:
|
38
|
-
- - "
|
38
|
+
- - "~>"
|
39
39
|
- !ruby/object:Gem::Version
|
40
|
-
version: '0'
|
40
|
+
version: '0.83'
|
41
41
|
- !ruby/object:Gem::Dependency
|
42
42
|
name: bundler
|
43
43
|
requirement: !ruby/object:Gem::Requirement
|
@@ -58,14 +58,14 @@ dependencies:
|
|
58
58
|
requirements:
|
59
59
|
- - "~>"
|
60
60
|
- !ruby/object:Gem::Version
|
61
|
-
version: '
|
61
|
+
version: '13.0'
|
62
62
|
type: :development
|
63
63
|
prerelease: false
|
64
64
|
version_requirements: !ruby/object:Gem::Requirement
|
65
65
|
requirements:
|
66
66
|
- - "~>"
|
67
67
|
- !ruby/object:Gem::Version
|
68
|
-
version: '
|
68
|
+
version: '13.0'
|
69
69
|
- !ruby/object:Gem::Dependency
|
70
70
|
name: yard
|
71
71
|
requirement: !ruby/object:Gem::Requirement
|
@@ -156,6 +156,7 @@ executables: []
|
|
156
156
|
extensions: []
|
157
157
|
extra_rdoc_files: []
|
158
158
|
files:
|
159
|
+
- ".circleci/config.yml"
|
159
160
|
- ".gitignore"
|
160
161
|
- Gemfile
|
161
162
|
- Gemfile.lock
|
@@ -163,7 +164,6 @@ files:
|
|
163
164
|
- LICENSE
|
164
165
|
- README.md
|
165
166
|
- Rakefile
|
166
|
-
- circle.yml
|
167
167
|
- danger-rubocop.gemspec
|
168
168
|
- lib/danger_plugin.rb
|
169
169
|
- lib/version.rb
|
@@ -192,8 +192,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
192
192
|
- !ruby/object:Gem::Version
|
193
193
|
version: '0'
|
194
194
|
requirements: []
|
195
|
-
|
196
|
-
rubygems_version: 2.7.6
|
195
|
+
rubygems_version: 3.0.8
|
197
196
|
signing_key:
|
198
197
|
specification_version: 4
|
199
198
|
summary: A Danger plugin for running Ruby files through Rubocop.
|
data/circle.yml
DELETED