netsoft-danger 0.3.4 → 0.3.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 896e582de631d926bd01d4f5d98d0bde7f20f9a0ec129600bcea3eff09ffbb53
-  data.tar.gz: 9613d84bcd13e28594d387b66c1d1cc379d2f6ee62857e4ddcf2bce30761466a
+  metadata.gz: ca4dcd0963df60b652e45e3bba8ef8262b350e52c0f659b5ec6660cf6f107ffa
+  data.tar.gz: 351f00d2d1f090c7a97e4385adc5c40af0284055b9d5a7676bdb1aec2bc99588
 SHA512:
-  metadata.gz: 9600eeb6dc24d45393f88ce2f4e956885209974d0910bec466354fca4950de21a78fad98986f2c1593074a44c72f95d8d0760254e6b08a5b3bd6b1a6daef5d39
-  data.tar.gz: 244fa8e69810e66bd44b37d6b156dc58229bc6ff6fe39f736928332f1209d200fc003651dc7c8f1863cc90d57dbb24f874e4075492239d33e3506829ecd752ec
+  metadata.gz: 6a29a7f5ffc53604bbbde1214b6ff006b7ead9d38cfee18fc843b2f5d8df467e1c77ed8e914895bc0e0a7e0eebb85808f5d5b5f1a23500c850c9738723ec5b4c
+  data.tar.gz: c7ceb9f06a2815b36ae58e540197aaa7866daa3168c741dcc4ea862eb52b3496612593e9c5ff3e7ea1791be89dc99cac995d66f1f401c4243bdcf30f27e48300

data/.circleci/config.yml

@@ -111,6 +111,10 @@ jobs:
           name: Build gem
           command: |-
             gem build *.gemspec
+      - run:
+          name: Run rubocop
+          command: |-
+            bundle exec rubocop
       - run:
           name: Run Danger
           command: |-
@@ -124,7 +128,9 @@ jobs:
           name: Deploy to gem server
           command: |-
             ./bin/tag_check.sh
+            ./bin/setup-rubygems.sh
             rm -rf pkg
             gem install geminabox
             rake build
             gem inabox -g ${HUBSTAFF_GEM_SERVER} pkg/*
+            gem push pkg/*.gem

data/.rubocop.yml

@@ -0,0 +1,15 @@
+inherit_gem:
+  netsoft-rubocop:
+    - default.yml
+
+Style/SignalException:
+  Exclude:
+    - Dangerfile
+
+Style/IfUnlessModifier:
+  Exclude:
+    - Dangerfile
+
+Layout/EmptyLineAfterGuardClause:
+  Exclude:
+    - Dangerfile

data/CHANGELOG.md

@@ -0,0 +1,38 @@
+# Changelog
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
+and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
+
+## [Next]
+### Added
+### Changed
+### Fixed
+
+## [0.3.9]
+### Fixed
+- correct API call to github for adding labels and fix logic in determining if we should or should not add the label
+
+## [0.3.8]
+### Added
+- auto add/remove some labels
+- allow conventional commit format for our migration, gemfile, and package.json
+- add support for product review labels
+- add support for renamed code review labels
+
+## [0.3.7]
+### Fixed
+- require English in the netsoft-circle bin
+
+## [0.3.6]
+### Added
+- rubocop checks for this gem
+### Changed
+- allow bundler 1.17.3
+
+## [0.3.5]
+### Added
+- package.json checks
+### Fixed
+- requie older version of faraday until octokit is fixed (https://github.com/octokit/octokit.rb/pull/1154)
+

data/Dangerfile

@@ -1,105 +1,156 @@
+# frozen_string_literal: true
+
+def toggle_label(github, label, should_set)
+  repo_name = github.pr_json['head']['repo']['full_name']
+  pr_number = github.pr_json['number']
+  has_label = github.pr_labels.include?(label)
+  puts repo_name: repo_name, pr_number: pr_number, labels: github.pr_labels, has_label: has_label, label: label, should_set: should_set
+  if should_set && !has_label
+    github.api.add_labels_to_an_issue(repo_name, pr_number, [label])
+  elsif !should_set && has_label
+    github.api.remove_label(repo_name, pr_number, label)
+  end
+end
+
 # Don't let testing shortcuts get into master by accident
-if Dir.exists?('spec')
-  fail("fdescribe left in tests") if `grep -r -e '\\bfdescribe\\b' spec/ |grep -v 'danger ok' `.length > 1
-  fail("fcontext left in tests") if `grep -r -e '\\bfcontext\\b' spec/ |grep -v 'danger ok' `.length > 1
-  fail("fit left in tests") if `grep -r -e '\\bfit\\b' spec/ | grep -v 'danger ok' `.length > 1
-  fail("ap left in tests") if `grep -r -e '\\bap\\b' spec/ | grep -v 'danger ok' `.length > 1
-  fail("puts left in tests") if `grep -r -e '\\bputs\\b' spec/ | grep -v 'danger ok' `.length > 1
+if Dir.exist?('spec')
+  fail('fdescribe left in tests') if `grep -r -e '\\bfdescribe\\b' spec/ |grep -v 'danger ok' `.length > 1
+  fail('fcontext left in tests') if `grep -r -e '\\bfcontext\\b' spec/ |grep -v 'danger ok' `.length > 1
+  fail('fit left in tests') if `grep -r -e '\\bfit\\b' spec/ | grep -v 'danger ok' `.length > 1
+  fail('ap left in tests') if `grep -r -e '\\bap\\b' spec/ | grep -v 'danger ok' `.length > 1
+  fail('puts left in tests') if `grep -r -e '\\bputs\\b' spec/ | grep -v 'danger ok' `.length > 1
 end
 
-if File.exists?('Gemfile')
+if File.exist?('Gemfile')
   if `grep -r -e "^ *gem 'hubstaff_[a-z]\\+" Gemfile | grep -e ",.\\+[a-zA-Z]" `.length > 1
-    fail("[gemfile] Beta hubstaff_* gems are not allowed in master/production")
+    fail('gemfile: Beta hubstaff_* gems are not allowed in master/production')
   end
   if `grep -r -e "^ *gem 'hubstaff_[a-z]\\+" Gemfile | grep -e ",.\\+'[~>=]\\+.\\+[a-zA-Z]" `.length > 1
-    fail("[gemfile] Beta hubstaff_* gems should be the exact version")
+    fail('gemfile: Beta hubstaff_* gems should be the exact version')
   end
   if `grep -r -e "^ *gem 'hubstaff_[a-z]\\+" Gemfile | grep -e ",.\\+[' ][0-9.]\\+'" | grep -v '~>' `.length > 1
-    fail("[gemfile] Release hubstaff_* gems should be using a ~> version")
+    fail('gemfile: Release hubstaff_* gems should be using a ~> version')
   end
 end
 
+if github.branch_for_head.start_with?('security')
+  toggle_label(github, 'security', true)
+end
+
+should_have_migration_label = false
+
 git.commits.each do |c|
-  short = " ( #{c.sha[0..7]} )"
-  has_migrations = c.diff_parent.any? {|f| f.path =~ /db\/migrate\// }
-  has_schema_changes = c.diff_parent.any? {|f| f.path =~ /db\/schema\.rb/ }
-  has_migration_msg = c.message =~ /^\[migration\]/
-  no_schema_ok = ENV['DANGER_NO_SCHEMA_OK'] || false
+  short              = " ( #{c.sha[0..7]} )"
+  has_migrations     = c.diff_parent.any? { |f| f.path =~ %r{db/migrate/} }
+  has_schema_changes = c.diff_parent.any? { |f| f.path =~ %r{db/schema\.rb} }
+  old_migration_msg  = c.message.start_with?('[migration]')
+  has_migration_msg  = old_migration_msg || c.message.match?(/\Amigration(\([A-Za-z]+\))?:/)
+  no_schema_ok       = ENV['DANGER_NO_SCHEMA_OK'] || false
   if has_migrations || has_schema_changes
     unless has_migration_msg
-      fail '[migration] Schema migration commits need to be prefixed with [migration]' + short
+      fail 'migration: Schema migration commits needs to be tagged with (migration). e.g. migration(Module): ' + short
+    end
+    if old_migration_msg
+      warn 'migration: Please switch to the new conventional commit format.'
     end
     if has_migrations && !has_schema_changes && !no_schema_ok
-      fail '[migration] Please checkin your schema.rb changes with your migration' + short
+      fail 'migration: Please checkin your schema.rb changes with your migration' + short
     end
     if !has_migrations && has_schema_changes
-      warn '[migration] Please checkin your migrations with your schema.rb changes' + short
+      warn 'migration: Please checkin your migrations with your schema.rb changes' + short
     end
-    if c.diff_parent.any? {|f| !( f.path =~ /db\/migrate\// or f.path =~ /db\/schema.rb/ ) }
-      fail '[migration] Migration commit contains non-migration changes' + short
+    if c.diff_parent.any? { |f| f.path !~ %r{db/migrate/|db/schema\.rb} }
+      fail 'migration: Migration commit contains non-migration changes' + short
     end
+
+    should_have_migration_label = true
   elsif has_migration_msg
     fail '[migration] Migration commit with no migrations!' + short
   end
 
-  has_hubstaff_icon_changes = c.diff_parent.any? {|f| f.path =~ /hubstaff(icons|font)/ || f.path =~ /fontcustom-manifest/ }
+  has_hubstaff_icon_changes = c.diff_parent.any? { |f| f.path =~ /hubstaff(icons|font)|fontcustom-manifest/ }
   if has_hubstaff_icon_changes
-    if c.diff_parent.any? {|f| !( f.path =~ /hubstaff-(icons|font)/ || f.path =~ /fontcustom-manifest/ ) }
-      fail '[hubstaff-icons] Put hubstaff-icon changes into their own commit' + short
+    if c.diff_parent.any? { |f| !(f.path =~ /hubstaff-(icons|font)/ || f.path =~ /fontcustom-manifest/) }
+      fail 'hubstaff-icons: Put hubstaff-icon changes into their own commit' + short
     end
   end
 
-  has_gemfile_changes = c.diff_parent.any? {|f| f.path =~ /Gemfile/ || f.path =~ /gemspec/ }
-  has_gemfile_msg = c.message =~ /^\[gemfile\]/
+  has_gemfile_changes = c.diff_parent.any? { |f| f.path =~ /Gemfile|gemspec/ }
+  old_gemfile_msg     = c.message.start_with?('[gemfile]')
+  has_gemfile_msg     = old_gemfile_msg || c.message.match?(/\Agemfile(\([A-Za-z]+\))?:/)
   if has_gemfile_changes
     unless has_gemfile_msg
-      fail '[gemfile] Gemfile commits need to be prefixed with [gemfile] ' + short
+      fail 'gemfile: Gemfile commits needs to be tagged with (gemfile). e.g. gemfile(Module): ' + short
     end
-    if c.diff_parent.any? {|f| !( f.path =~ /Gemfile/ || f.path =~ /gemspec/ ) }
-      fail '[gemfile] Gemfile commit contains non-gemfile changes' + short
+    if old_migration_msg
+      warn 'gemfile: Please switch to the new conventional commit format.'
     end
-    if c.diff_parent.any? {|f| f.path == 'Gemfile.lock' }
-      unless `grep -e '^   1.15.2$' Gemfile.lock`.length > 1
-        fail("[gemfile] Gemfile not bundled with bundler 1.15.2")
+    if c.diff_parent.any? { |f| f.path !~ /Gemfile|gemspec/ }
+      fail 'gemfile: Gemfile commit contains non-gemfile changes' + short
+    end
+    if c.diff_parent.any? { |f| f.path == 'Gemfile.lock' }
+      unless `grep -E -- '^BUNDLED WITH\s*\n\s+(1\\.15\\.2|1\\.17\\.3)$' Gemfile.lock`.length > 1
+        fail('gemfile: Gemfile not bundled with bundler 1.15.2 or 1.17.3')
       end
     end
   elsif has_gemfile_msg
-    fail '[gemfile] Gemfile commit has no gemfile changes!' + short
+    fail 'gemfile: Gemfile commit has no gemfile changes!' + short
+  end
+
+  has_package_changes = c.diff_parent.any? { |f| f.path =~ /package\.json|yarn\.lock/ }
+  old_package_msg     = c.message.start_with?('[package.json]')
+  has_package_msg     = old_package_msg || c.message.match?(/\Apackage(\([A-Za-z]+\))?:/)
+  if has_package_changes
+    unless has_package_msg
+      fail 'package: Package.json commits needs to be tagged with package. e.g package(Module): ' + short
+    end
+    if old_package_msg
+      warn 'package: Please switch to the new conventional commit format.'
+    end
+    if c.diff_parent.any? { |f| f.path !~ /package\.json|yarn\.lock/ }
+      fail 'package: Package.json commit contains non-package changes' + short
+    end
+  elsif has_package_msg
+    fail 'package: Pacakge.json commit has no package changes!' + short
   end
 end
 
-require 'open-uri'
-
-artifact_url = "https://circleci.com/api/v1.1/project/github/#{ENV['CIRCLE_PROJECT_USERNAME']}/#{ENV["CIRCLE_PROJECT_REPONAME"]}/#{ENV['CIRCLE_BUILD_NUM']}/artifacts?circle-token=#{ENV['CIRCLE_TOKEN']}"
-artifacts = JSON.load(open(artifact_url).read).map{|a| a["url"]}
-
-jest = artifacts.find{ |artifact| artifact.end_with?('jest/index.html') }
-coverage = artifacts.find{ |artifact| artifact.end_with?('coverage/index.html') }
-rubocop = artifacts.find{ |artifact| artifact.end_with?('rubocop/report.html') }
-eslint = artifacts.find{ |artifact| artifact.end_with?('eslint/report.html') }
-rspec_files = artifacts.select{ |artifact| artifact =~ /rspec-(.+)\.html$/ }
-
-{}.tap do |hash|
-  hash['Ruby coverage report'] = coverage if coverage
-  hash['RSpec test report'] = rspec_files unless rspec_files.empty?
-  hash['RuboCop inspection report'] = rubocop if rubocop
-  hash['ESLint inspection report'] = eslint if eslint
-  hash['Jest coverage report'] = jest if jest
-end.each do |msg, links|
-  links = [*links]
-  if links.size == 1
-    message("[#{msg}](#{links[0]})")
-  else
-    r         = /rspec-(.+)\.html$/
-    the_links = links.map do |l|
-      m = r.match(l)
-      if m
-        "[#{m[1]}](#{l})"
-      else
-        "[link](#{l})"
-      end
-    end.join(', ')
+toggle_label(github, 'run migration', should_have_migration_label)
 
-    message("#{msg} - #{the_links}")
+if ENV['CIRCLE_TOKEN']
+  require 'open-uri'
+
+  artifact_url = "https://circleci.com/api/v1.1/project/github/#{ENV['CIRCLE_PROJECT_USERNAME']}/#{ENV['CIRCLE_PROJECT_REPONAME']}/#{ENV['CIRCLE_BUILD_NUM']}/artifacts?circle-token=#{ENV['CIRCLE_TOKEN']}"
+  artifacts    = JSON.parse(URI.parse(artifact_url).read).map { |a| a['url'] }
+
+  jest        = artifacts.find { |artifact| artifact.end_with?('jest/index.html') }
+  coverage    = artifacts.find { |artifact| artifact.end_with?('coverage/index.html') }
+  rubocop     = artifacts.find { |artifact| artifact.end_with?('rubocop/report.html') }
+  eslint      = artifacts.find { |artifact| artifact.end_with?('eslint/report.html') }
+  rspec_files = artifacts.select { |artifact| artifact =~ /rspec-(.+)\.html$/ }
+
+  {}.tap do |hash|
+    hash['Ruby coverage report']      = coverage if coverage
+    hash['RSpec test report']         = rspec_files unless rspec_files.empty?
+    hash['RuboCop inspection report'] = rubocop if rubocop
+    hash['ESLint inspection report']  = eslint if eslint
+    hash['Jest coverage report']      = jest if jest
+  end.each do |msg, links|
+    links = [*links]
+    if links.size == 1
+      message("[#{msg}](#{links[0]})")
+    else
+      r         = /rspec-(.+)\.html$/
+      the_links = links.map { |l|
+        m = r.match(l)
+        if m
+          "[#{m[1]}](#{l})"
+        else
+          "[link](#{l})"
+        end
+      }.join(', ')
+
+      message("#{msg} - #{the_links}")
+    end
   end
 end

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gemspec

data/Rakefile

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 require 'bundler'
 Bundler::GemHelper.install_tasks
 
 desc 'Default: run build.'
-task :default => :build
+task default: :build

data/bin/netsoft-circle

@@ -1,35 +1,36 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require 'thor'
 require 'fileutils'
+require 'English'
 
-class NetsoftCircle < Thor
+class NetsoftCircle < Thor # :nodoc:
   desc 'setup', 'Setup Heroku for deployment'
   def setup
-
     File.open("#{ENV['HOME']}/.netrc", 'a+') do |file|
-      file << <<-EOF
+      file << <<~CONFIG
 
-machine api.heroku.com
-  login #{ENV['HEROKU_LOGIN']}
-  password #{ENV['HEROKU_API_KEY']}
-machine git.heroku.com
-  login #{ENV['HEROKU_LOGIN']}
-  password #{ENV['HEROKU_API_KEY']}
-EOF
+        machine api.heroku.com
+          login #{ENV['HEROKU_LOGIN']}
+          password #{ENV['HEROKU_API_KEY']}
+        machine git.heroku.com
+          login #{ENV['HEROKU_LOGIN']}
+          password #{ENV['HEROKU_API_KEY']}
+      CONFIG
     end
 
-    Dir.mkdir("#{ENV['HOME']}/.ssh") unless Dir.exists?("#{ENV['HOME']}/.ssh")
+    Dir.mkdir("#{ENV['HOME']}/.ssh") unless Dir.exist?("#{ENV['HOME']}/.ssh")
     File.open("#{ENV['HOME']}/.ssh/config", 'a+') do |file|
-      file << <<-EOF
+      file << <<~CONFIG
 
-VerifyHostKeyDNS yes
-StrictHostKeyChecking no
-EOF
+        VerifyHostKeyDNS yes
+        StrictHostKeyChecking no
+      CONFIG
     end
 
     system('git config --global url."https://git.heroku.com/".insteadOf heroku:')
-    exit(1) unless $?.success?
+    exit(1) unless $CHILD_STATUS.success?
   end
 
   desc 'merge', 'Merges several simplecov json result files'
@@ -37,6 +38,7 @@ EOF
   def merge(*files)
     require 'simplecov'
     return if files.empty?
+
     results = []
 
     files.each do |file|
@@ -54,7 +56,7 @@ EOF
 
   desc 'rspec', 'Run rspec'
   def rspec
-    system <<-EOF
+    system <<~COMMAND
       bundle _${BUNDLE_VERSION}_ exec rspec \
         --color \
         --format RspecJunitFormatter \
@@ -63,20 +65,20 @@ EOF
         --out $CIRCLE_ARTIFACTS/rspec.html \
         --format progress \
         $(circleci tests glob "spec/**/*_spec.rb" | circleci tests split --split-by=timings)
-    EOF
-    exit(1) unless $?.success?
+    COMMAND
+    exit(1) unless $CHILD_STATUS.success?
   end
 
   desc 'rubocop', 'Run rubocop'
   def rubocop
-    system <<-EOF
+    system <<~COMMAND
       bundle _${BUNDLE_VERSION}_ exec rubocop \
         --parallel \
         --format progress \
         --format html \
         --out $CIRCLE_ARTIFACTS/rubocop/report.html
-    EOF
-    exit(1) unless $?.success?
+    COMMAND
+    exit(1) unless $CHILD_STATUS.success?
   end
 
   def self.exit_on_failure?

data/bin/setup-rubygems.sh

@@ -0,0 +1,3 @@
+mkdir ~/.gem
+echo -e "---\n:rubygems_api_key: $RUBYGEMS_API_KEY" > ~/.gem/credentials
+chmod 0600 ~/.gem/credentials

data/dangerfiles/pr.js

@@ -15,8 +15,12 @@ if ((danger.github.pr.body || '').length < 5) {
   fail("Please provide a summary in the Pull Request description");
 }
 
-if (!labels.includes('review passed')) {
-  fail("Has not passed code-review");
+if (labels.includes('product review needed')) {
+  fail('Has not passed product review');
+}
+
+if (!(labels.includes('review passed') || labels.includes('code review passed'))) {
+  fail("Has not passed code review");
 }
 
 if (!labels.includes('QA passed')) {

data/lib/netsoft-danger/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module NetsoftDanger
-  VERSION = '0.3.4'.freeze
+  VERSION = '0.3.9'
 end