danger-go 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 922a2aadc91e2b58508bf4cc19482dac1c51dec7
-  data.tar.gz: 9c723007bec4dfc3621fea94c9a04b3087c49895
+  metadata.gz: d5dad39736be1112c44189d56b278b5cd7557107
+  data.tar.gz: 5c7dcac83c5884358e7e3a52d41b4620dafbe455
 SHA512:
-  metadata.gz: 02ed62b8ea45a738eba78089563461d79f1e7d24e53bcc117c3eaf8226db699569091df22b7287fe9d93ba00ad30aa92ce7aeef7f2d1ea163ceb5247bb7d3fb5
-  data.tar.gz: 5fbb98e324a123123b384e8ef447697051c0f04174ce4717bae793c852f4450e1f900a903939893de62926171cba51b663d9be78b1a1b5949b27890d7502833a
+  metadata.gz: 2bec86796c5c8abe415b7bfdf424b18269a1da6424c999d88bd7942f0c5dda0cd149e364bb3ca154d091ef3659d0fcf29394f985e9f65225a8864a8c5f8d55b4
+  data.tar.gz: 496e672811e22ab4ac20efe8193fa4b095381c604e72fa8eb10ab302e33e864226d895c2551953a53c10e5c4805d6b1c934cee3859631f262ada22ab9a56a8b5

data/.gitignore

@@ -13,3 +13,9 @@
 
 # Output of the go coverage tool, specifically when used with LiteIDE
 *.out
+
+# Gem file
+*.gem
+
+# Rake
+pkg/

data/.rubocop.yml

@@ -0,0 +1,22 @@
+AllCops:
+  Exclude:
+    - 'bin/**/*'
+    - 'db/**/*'
+    - 'log/**/*'
+    - 'tmp/**/*'
+    - 'vendor/**/*'
+    - 'Guardfile'
+  DisplayCopNames: true
+
+Metrics/LineLength:
+  Max: 120
+
+SpaceAroundEqualsInParameterDefault:
+  Enabled: false
+
+Metrics/BlockLength:
+  Exclude:
+    - 'Rakefile'
+    - '**/*.rake'
+    - 'spec/**/*.rb'
+    - 'test/**/*.rb'

data/.travis.yml

@@ -0,0 +1,21 @@
+language: ruby
+
+rvm:
+  - 2.6.0
+  - 2.5.3
+  - 2.4.5
+  - 2.3.8
+
+bundler_args: "--without documentation --path bundle"
+
+install:
+  - gem install bundler -v '2.0.1'
+  - bundle install
+
+after_install:
+  - rake install
+
+script:
+  - bundle exec rake spec
+
+sudo: required

data/.yardoc/checksums

@@ -0,0 +1,4 @@
+/Users/user/Desktop/danger-go/lib/danger_golint.rb e829c86e3321bf181656d19e46f386ca1fa9d2e6
+/Users/user/Desktop/danger-go/lib/danger_plugin.rb 91e59d5cc142d21c5e68e0be029e0879a93bb5f4
+/Users/user/Desktop/danger-go/lib/golint/gem_version.rb 3dc8e67f262afb3c0aebb0505e03057dc38e42e7
+/Users/user/Desktop/danger-go/lib/golint/plugin.rb 767849f952b5462367f867d11dd3fcaed5ff95e3

data/Gemfile

@@ -1,4 +1,16 @@
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in danger-golint.gemspec
+source "https://rubygems.org"
+
 gemspec
+
+gem "bundler"
+gem "rake", "~> 10.0"
+gem "guard", "~> 2.14"
+gem "guard-rspec", "~> 4.7"
+gem "listen", "3.0.7"
+gem "pry", "~> 0.10"
+gem "rspec", "~> 3.4"
+gem "rubocop", "~> 0.49.0"
+gem "yard", "~> 0.9.11"

data/Gemfile.lock

@@ -9,42 +9,43 @@ GEM
   specs:
     addressable (2.5.2)
       public_suffix (>= 2.0.2, < 4.0)
-    ast (2.3.0)
+    ast (2.4.0)
     claide (1.0.2)
     claide-plugins (0.9.2)
       cork
       nap
       open4 (~> 1.3)
-    coderay (1.1.1)
+    coderay (1.1.2)
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (5.5.5)
+    danger (6.0.4)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
       faraday (~> 0.9)
       faraday-http-cache (~> 1.0)
-      git (~> 1)
-      kramdown (~> 1.5)
+      git (~> 1.5)
+      kramdown (~> 2.0)
+      kramdown-parser-gfm (~> 1.0)
       no_proxy_fix
       octokit (~> 4.7)
       terminal-table (~> 1)
     danger-plugin-api (1.0.0)
       danger (> 2.0)
-    diff-lcs (1.2.5)
-    faraday (0.13.1)
+    diff-lcs (1.3)
+    faraday (0.15.4)
       multipart-post (>= 1.2, < 3)
     faraday-http-cache (1.3.1)
       faraday (~> 0.8)
-    ffi (>= 1.9.24)
+    ffi (1.10.0)
     formatador (0.2.5)
-    git (1.3.0)
-    guard (2.14.0)
+    git (1.5.0)
+    guard (2.15.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
-      lumberjack (~> 1.0)
+      lumberjack (>= 1.0.12, < 2.0)
       nenv (~> 0.1)
       notiffany (~> 0.0)
       pry (>= 0.9.12)
@@ -55,12 +56,14 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    kramdown (1.16.2)
+    kramdown (2.1.0)
+    kramdown-parser-gfm (1.0.1)
+      kramdown (~> 2.0)
     listen (3.0.7)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9.7)
-    lumberjack (1.0.10)
-    method_source (0.8.2)
+    lumberjack (1.0.13)
+    method_source (0.9.2)
     multipart-post (2.0.0)
     nap (1.1.0)
     nenv (0.3.0)
@@ -68,37 +71,36 @@ GEM
     notiffany (0.1.1)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (4.8.0)
+    octokit (4.14.0)
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
-    parallel (1.12.1)
-    parser (2.4.0.2)
-      ast (~> 2.3)
-    powerpack (0.1.1)
-    pry (0.10.4)
+    parallel (1.17.0)
+    parser (2.6.2.1)
+      ast (~> 2.4.0)
+    powerpack (0.1.2)
+    pry (0.12.2)
       coderay (~> 1.1.0)
-      method_source (~> 0.8.1)
-      slop (~> 3.4)
-    public_suffix (2.0.5)
+      method_source (~> 0.9.0)
+    public_suffix (3.0.3)
     rainbow (2.2.2)
       rake
     rake (10.5.0)
-    rb-fsevent (0.9.8)
-    rb-inotify (0.9.7)
-      ffi (>= 1.9.24)
-    rspec (3.5.0)
-      rspec-core (~> 3.5.0)
-      rspec-expectations (~> 3.5.0)
-      rspec-mocks (~> 3.5.0)
-    rspec-core (3.5.4)
-      rspec-support (~> 3.5.0)
-    rspec-expectations (3.5.0)
+    rb-fsevent (0.10.3)
+    rb-inotify (0.10.0)
+      ffi (~> 1.0)
+    rspec (3.8.0)
+      rspec-core (~> 3.8.0)
+      rspec-expectations (~> 3.8.0)
+      rspec-mocks (~> 3.8.0)
+    rspec-core (3.8.0)
+      rspec-support (~> 3.8.0)
+    rspec-expectations (3.8.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.5.0)
-    rspec-mocks (3.5.0)
+      rspec-support (~> 3.8.0)
+    rspec-mocks (3.8.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.5.0)
-    rspec-support (3.5.0)
+      rspec-support (~> 3.8.0)
+    rspec-support (3.8.0)
     rubocop (0.49.1)
       parallel (~> 1.10)
       parser (>= 2.3.3.1, < 3.0)
@@ -106,28 +108,27 @@ GEM
       rainbow (>= 1.99.1, < 3.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.9.0)
+    ruby-progressbar (1.10.0)
     sawyer (0.8.1)
       addressable (>= 2.3.5, < 2.6)
       faraday (~> 0.8, < 1.0)
     shellany (0.0.1)
-    slop (3.6.0)
     terminal-table (1.8.0)
       unicode-display_width (~> 1.1, >= 1.1.1)
-    thor (0.19.1)
-    unicode-display_width (1.1.1)
-    yard (0.9.12)
+    thor (0.20.3)
+    unicode-display_width (1.5.0)
+    yard (0.9.19)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  bundler (~> 1.3)
+  bundler
   danger-go!
   guard (~> 2.14)
   guard-rspec (~> 4.7)
   listen (= 3.0.7)
-  pry
+  pry (~> 0.10)
   rake (~> 10.0)
   rspec (~> 3.4)
   rubocop (~> 0.49.0)

data/README.md

@@ -1,7 +1,8 @@
 <h1 align="center">danger-go</h1>
 <p align="center">
-    <a href="https://github.com/KennethanCeyer/danger-go/blob/master/LICENSE" target="_blank"><img src="http://img.shields.io/badge/license-MIT-green.svg?style=flat" alt="License"></a>
+    <a href="https://travis-ci.com/KennethanCeyer/danger-go" target="_blank"><img src="https://travis-ci.com/KennethanCeyer/danger-go.svg?branch=master" alt="Build Status"></a>
     <a href="http://rubygems.org/gems/danger-go" target="_blank"><img src="https://img.shields.io/gem/v/danger-go.svg?style=flat" alt="License"></a>
+    <a href="https://github.com/KennethanCeyer/danger-go/blob/master/LICENSE" target="_blank"><img src="http://img.shields.io/badge/license-MIT-green.svg?style=flat" alt="License"></a>
 </p>
 <p align="center">A Dangerfile for golang developer (golint)</p>
 <p align="center"><img src="./docs/images/gopher.png"></p>

data/lib/golint/gem_version.rb

@@ -1,3 +1,3 @@
 module Golint
-  VERSION = '0.0.1'.freeze
+  VERSION = '0.0.2'.freeze
 end

data/lib/golint/plugin.rb

@@ -18,13 +18,13 @@ module Danger
     # @return [String]
     attr_writer :base_dir
 
-    MARKDOWN_TEMPLATE = ""\
-      "## DangerGo found issues\n\n"\
-      "| File | Line | Column | Reason |\n"\
-      "|------|------|--------|--------|\n"\
+    MARKDOWN_TEMPLATE = \
+      '## DangerGo found issues\n\n'\
+      '| File | Line | Column | Reason |\n'\
+      '|------|------|--------|--------|\n'.freeze
 
     def base_dir
-      @base_dir || "."
+      @base_dir || '.'
     end
 
     # Lints go files.
@@ -34,7 +34,7 @@ module Danger
     # @return  [void]
     #
     def lint
-      errors = lint_results.reject { |r| r.nil? }
+      errors = lint_results.reject(&:nil?)
 
       return if errors.empty?
 
@@ -56,7 +56,7 @@ module Danger
     def lint_results
       bin = 'golint'
       system 'go get -u golang.org/x/lint/golint' unless golint_installed?
-      return run_lint(bin, base_dir)
+      run_lint(bin, base_dir)
     end
 
     # Run golint aginst a single dir.
@@ -69,8 +69,7 @@ module Danger
     #
     # @return [Output]
     def run_lint(bin, dir)
-      command = "#{bin}"
-      `#{command} #{dir}`.split('\n')
+      `#{bin} #{dir}`.split('\n')
     end
 
     # Print markdown string
@@ -80,15 +79,15 @@ module Danger
     # @return  [string]
     def print_markdown_table(errors=[])
       report = errors.inject(MARKDOWN_TEMPLATE) do |out, error_line|
-        file, line, column, reason = error_line.split(":")
-        out += "| #{short_link(file, line)} | #{line} | #{column} | #{reason.strip.gsub("'", "`")} |\n"
+        file, line, column, reason = error_line.split(':')
+        out + "| #{short_link(file, line)} | #{line} | #{column} | #{reason.strip.tr('\'', '`')} |\n"
       end
 
       markdown(report)
     end
 
     def short_link(file, line)
-      if danger.scm_provider.to_s == "github"
+      if danger.scm_provider.to_s == 'github'
         return github.html_link("#{file}#L#{line}", full_path: false)
       end
 

data/spec/golint_spec.rb

@@ -17,30 +17,30 @@ module Danger
 
       context 'golint not installed' do
         before do
-          allow(@golint).to receive(:`).with("which golint").and_return("")
+          allow(@golint).to receive(:`).with('which golint').and_return('')
         end
       end
 
       context 'golint installed' do
         before do
-          allow(@golint).to receive(:`).with("which golint").and_return("/usr/bin/golint")
+          allow(@golint).to receive(:`).with('which golint').and_return('/usr/bin/golint')
         end
 
         describe 'lint' do
           it 'runs lint from current directory by default' do
-            expect(@golint).to receive(:`).with("golint .").and_return("")
+            expect(@golint).to receive(:`).with('golint .').and_return('')
             @golint.lint
           end
 
           it 'runs lint from a custom directory' do
-            expect(@golint).to receive(:`).with("golint my/custom/directory").and_return("")
+            expect(@golint).to receive(:`).with('golint my/custom/directory').and_return('')
 
-            @golint.base_dir = "my/custom/directory"
+            @golint.base_dir = 'my/custom/directory'
             @golint.lint
           end
 
           it 'handles a lint with no errors' do
-            allow(@golint).to receive(:`).with("golint .").and_return("")
+            allow(@golint).to receive(:`).with('golint .').and_return('')
             @golint.lint
             expect(@golint.status_report[:markdowns].first).to be_nil
           end
@@ -48,19 +48,20 @@ module Danger
 
         context 'when running on github' do
           it 'handles a lint with errors count greater than threshold' do
-            lint_report = "test.go:213:3: don't use underscores in Go names; var my_var should be myVar\n"
+            lint_report = 'test.go:213:3: don\'t use underscores in Go names; var my_var should be myVar\n'
 
-            allow(@golint).to receive(:`).with("golint .").and_return(lint_report)
-            allow(@dangerfile.danger).to receive(:scm_provider).and_return("github")
+            allow(@golint).to receive(:`).with('golint .').and_return(lint_report)
+            allow(@dangerfile.danger).to receive(:scm_provider).and_return('github')
             allow(@dangerfile.github).to receive(:html_link)
-                                             .with("test.go#L213", full_path: false)
-                                             .and_return("fake_link_to:test.go#L213")
+              .with('test.go#L213', full_path: false)
+              .and_return('fake_link_to:test.go#L213')
 
             @golint.lint
 
             markdown = @golint.status_report[:markdowns].first
-            expect(markdown.message).to include("## DangerGo found issues")
-            expect(markdown.message).to include("| fake_link_to:test.go#L213 | 213 | 3 | don`t use underscores in Go names; var my_var should be myVar |")
+            expect(markdown.message).to include('## DangerGo found issues')
+            expect(markdown.message).to include('| fake_link_to:test.go#L213 | 213 | 3 | ' \
+            'don`t use underscores in Go names; var my_var should be myVar |')
           end
         end
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger-go
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - KennethanCeyer
@@ -158,6 +158,13 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rubocop.yml"
+- ".travis.yml"
+- ".yardoc/checksums"
+- ".yardoc/complete"
+- ".yardoc/object_types"
+- ".yardoc/objects/root.dat"
+- ".yardoc/proxy_types"
 - Gemfile
 - Gemfile.lock
 - Guardfile