Merge pull request #101 from github/kpaulisse-puppet-4-10

Add puppet 4.10 to the CI rotation and update gems

Author: kpaulisse

.rubocop.yml

@@ -22,6 +22,9 @@ Metrics/AbcSize:
   Max: 100
   Severity: warning
 
+Metrics/BlockLength:
+  Enabled: false
+
 Metrics/ClassLength:
   Max: 500
   Severity: warning
@@ -40,12 +43,33 @@ Metrics/PerceivedComplexity:
 
 # Indentation
 CaseIndentation:
-  IndentWhenRelativeTo: end
   IndentOneStep: false
 
 Lint/EndAlignment:
-  AlignWith: variable
+  EnforcedStyleAlignWith: variable
 
 # To let us use a '-' in the gem name
 Style/FileName:
   Enabled: false
+
+# To fix later
+Style/PercentLiteralDelimiters:
+  Enabled: false
+Style/VariableNumber:
+  Enabled: false
+Security/YAMLLoad:
+  Enabled: false
+Lint/AmbiguousBlockAssociation:
+  Enabled: false
+Lint/UnifiedInteger:
+  Enabled: false
+Style/GuardClause:
+  Enabled: false
+Style/EmptyMethod:
+  Enabled: false
+Style/SymbolArray:
+  Enabled: false
+Style/IndentHeredoc:
+  Enabled: false
+Style/ExtraSpacing:
+  Enabled: false

octocatalog-diff.gemspec

@@ -1,6 +1,6 @@
 require_relative 'lib/octocatalog-diff/version'
 
-DEFAULT_PUPPET_VERSION = '4.5.3'.freeze
+DEFAULT_PUPPET_VERSION = '4.10.0'.freeze
 
 Gem::Specification.new do |s|
   s.required_ruby_version = '>= 2.0.0'
@@ -26,19 +26,19 @@ EOF
   s.add_runtime_dependency 'diffy', '>= 3.1.0'
   s.add_runtime_dependency 'httparty', '>= 0.11.0'
   s.add_runtime_dependency 'hashdiff', '>= 0.3.0'
-  s.add_runtime_dependency 'parallel', '>= 1.9.0'
+  s.add_runtime_dependency 'parallel', '>= 1.11.1'
   s.add_runtime_dependency 'rugged', '>= 0.25.0b2'
 
   s.add_development_dependency 'rspec', '~> 3.4.0'
   s.add_development_dependency 'rake', '11.2.2'
   s.add_development_dependency 'parallel_tests', '2.7.1'
   s.add_development_dependency 'rspec-retry', '0.5.0'
 
-  s.add_development_dependency 'rubocop', '~> 0.35'
+  s.add_development_dependency 'rubocop', '= 0.48.1'
 
   s.add_development_dependency 'puppetdb-terminus', '3.2.4'
 
-  s.add_development_dependency 'simplecov'
+  s.add_development_dependency 'simplecov', '>= 0.14.1'
   s.add_development_dependency 'simplecov-json'
 
   if ENV['PUPPET_VERSION']

script/cibuild

@@ -6,22 +6,16 @@
 # with one or more Puppet versions, with PUPPET_VERSIONS set to a space-separated list
 # of versions to test.
 
-[ -z "$PUPPET_VERSIONS" ] && export PUPPET_VERSIONS='3.8.7 4.5.3'
+[ -z "$PUPPET_VERSIONS" ] && export PUPPET_VERSIONS='3.8.7 4.5.3 4.10.0'
 [ -z "$RUBOCOP_TEST" ] && export RUBOCOP_TEST='true'
 [ -z "$RSPEC_TEST" ] && export RSPEC_TEST='true'
 
-# FIXME: Needed only for Janky. Remove for travis.
-if env | grep ^JANKY_ >/dev/null 2>&1; then
-  export PATH=/usr/share/rbenv/shims:$PATH
-  export RBENV_VERSION=2.1.2-github
-fi
-
 echo 'Starting script/cibuild'
 
 DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && cd .. && pwd )"
 
 # Create a temporary file to capture output of various steps.
-export OUTPUT_FILE=$(mktemp)
+export OUTPUT_FILE="$(mktemp)"
 function cleanup() {
   rm -rf "$OUTPUT_FILE"
 }
@@ -79,12 +73,12 @@ if [ "$RSPEC_TEST" = "true" ]; then
 
     # Make sure `script/puppet` returns the desired puppet version
     pv_test=$( "${DIR}/script/puppet" --version 2>&1 )
-    if [ $? -eq 0 -a "$pv_test" = "$pv" ]; then
+    if [ $? -eq 0 ] && [ "$pv_test" = "$pv" ]; then
       echo "Confirmed Puppet version = ${pv_test}"
     else
       echo "Failures:"
       echo "- Unable to confirm that Puppet version = ${pv}"
-      echo $pv_test
+      echo "$pv_test"
       RSPEC_EXITCODE=255
     fi
 
@@ -108,4 +102,4 @@ fi
 echo "Finished script/cibuild:"
 [ "$RUBOCOP_EXITCODE" -ge 0 ] && echo "  - rubocop: exit ${RUBOCOP_EXITCODE}"
 [ "$RSPEC_EXITCODE" -ge 0 ] && echo "  - rspec:   exit ${RSPEC_EXITCODE}"
-if [ "$RUBOCOP_EXITCODE" -gt 0 -o "$RSPEC_EXITCODE" -gt 0 ]; then exit 1; else exit 0; fi
+if [ "$RUBOCOP_EXITCODE" -gt 0 ] || [ "$RSPEC_EXITCODE" -gt 0 ]; then exit 1; else exit 0; fi