danger-swiftlint 0.1.1 → 0.1.2
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/Changelog.md +4 -0
- data/README.md +1 -1
- data/lib/danger_plugin.rb +3 -0
- data/lib/version.rb +1 -1
- data/spec/danger_plugin_spec.rb +4 -4
- metadata +2 -2
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA1:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 724a5435fe3b3b09ef18f15e76407a5f9d2deea0
|
|
4
|
+
data.tar.gz: ad2d97395e8578039090fdf1898b8f290f6b1ff8
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: c181f89260aa3c63f1efae0b2a63312025a0bc23cf6471a207c8a6c6f0296129bcc5434c3129211d661e729d75cdcc89a88c99a63bd7da07762644673cd9cbb8
|
|
7
|
+
data.tar.gz: 5b6916bb7e8011aaff00bb19b96ccf30cd80c7f8cdd9572ee9ac4a92a53908e4e7a340511155c778831e77fb1937b19962b971a3b67326a1e216daf1a4e592c6
|
data/Changelog.md
CHANGED
|
@@ -4,6 +4,10 @@
|
|
|
4
4
|
|
|
5
5
|
- Nothing yet!
|
|
6
6
|
|
|
7
|
+
## 0.1.2
|
|
8
|
+
|
|
9
|
+
- Adds support for files with spaces in their names. See [#9](https://github.com/ashfurrow/danger-swiftlint/issues/9).
|
|
10
|
+
|
|
7
11
|
## 0.1.1
|
|
8
12
|
|
|
9
13
|
- Fixes double-escaped newline characters. See [#11](https://github.com/ashfurrow/danger-swiftlint/issues/11).
|
data/README.md
CHANGED
|
@@ -30,7 +30,7 @@ swiftlint.lint_files
|
|
|
30
30
|
And finally, you can provide a list of files manually:
|
|
31
31
|
|
|
32
32
|
``` ruby
|
|
33
|
-
# Look through all changed
|
|
33
|
+
# Look through all changed Swift files
|
|
34
34
|
swift_files = (modified_files + added_files).select do |file|
|
|
35
35
|
file.end_with?(".swift")
|
|
36
36
|
end
|
data/lib/danger_plugin.rb
CHANGED
|
@@ -40,6 +40,9 @@ module Danger
|
|
|
40
40
|
swift_files = files ? Dir.glob(files) : (modified_files + added_files)
|
|
41
41
|
swift_files.select! do |line| line.end_with?(".swift") end
|
|
42
42
|
|
|
43
|
+
# Make sure we don't fail when paths have spaces
|
|
44
|
+
swift_files = swift_files.map { |file| "\"#{file}\"" }
|
|
45
|
+
|
|
43
46
|
swiftlint_command = "swiftlint lint --quiet --reporter json"
|
|
44
47
|
swiftlint_command += " --config #{config_file}" if config_file
|
|
45
48
|
|
data/lib/version.rb
CHANGED
data/spec/danger_plugin_spec.rb
CHANGED
|
@@ -5,7 +5,7 @@ module Danger
|
|
|
5
5
|
it 'is a plugin' do
|
|
6
6
|
expect(Danger::DangerSwiftlint < Danger::Plugin).to be_truthy
|
|
7
7
|
end
|
|
8
|
-
|
|
8
|
+
|
|
9
9
|
describe 'with Dangerfile' do
|
|
10
10
|
before do
|
|
11
11
|
@dangerfile = testing_dangerfile
|
|
@@ -34,7 +34,7 @@ module Danger
|
|
|
34
34
|
end
|
|
35
35
|
|
|
36
36
|
it 'handles a known SwiftLint report' do
|
|
37
|
-
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --path spec/fixtures/SwiftFile.swift').and_return(@swiftlint_response)
|
|
37
|
+
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --path "spec/fixtures/SwiftFile.swift"').and_return(@swiftlint_response)
|
|
38
38
|
|
|
39
39
|
# Do it
|
|
40
40
|
@swiftlint.lint_files("spec/fixtures/*.swift")
|
|
@@ -52,7 +52,7 @@ module Danger
|
|
|
52
52
|
it 'handles no files' do
|
|
53
53
|
allow(@swiftlint).to receive(:modified_files).and_return(['spec/fixtures/SwiftFile.swift'])
|
|
54
54
|
allow(@swiftlint).to receive(:added_files).and_return([])
|
|
55
|
-
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --path spec/fixtures/SwiftFile.swift').and_return(@swiftlint_response)
|
|
55
|
+
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --path "spec/fixtures/SwiftFile.swift"').and_return(@swiftlint_response)
|
|
56
56
|
|
|
57
57
|
@swiftlint.lint_files
|
|
58
58
|
|
|
@@ -61,7 +61,7 @@ module Danger
|
|
|
61
61
|
|
|
62
62
|
it 'uses a config file' do
|
|
63
63
|
@swiftlint.config_file = 'some_config.yml'
|
|
64
|
-
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --config some_config.yml --path spec/fixtures/SwiftFile.swift').and_return(@swiftlint_response)
|
|
64
|
+
allow(@swiftlint).to receive(:`).with('swiftlint lint --quiet --reporter json --config some_config.yml --path "spec/fixtures/SwiftFile.swift"').and_return(@swiftlint_response)
|
|
65
65
|
|
|
66
66
|
@swiftlint.lint_files("spec/fixtures/*.swift")
|
|
67
67
|
|
metadata
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: danger-swiftlint
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.1.
|
|
4
|
+
version: 0.1.2
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- Ash Furrow
|
|
@@ -10,7 +10,7 @@ authors:
|
|
|
10
10
|
autorequire:
|
|
11
11
|
bindir: bin
|
|
12
12
|
cert_chain: []
|
|
13
|
-
date: 2016-07-
|
|
13
|
+
date: 2016-07-12 00:00:00.000000000 Z
|
|
14
14
|
dependencies:
|
|
15
15
|
- !ruby/object:Gem::Dependency
|
|
16
16
|
name: danger
|