Merge pull request #36 from zendesk/nony--cleanup-ci

Drop Ruby 2.4 and make CI simpler

Author: HeyNonster

.github/workflows/ci.yml

@@ -8,27 +8,36 @@ jobs:
     strategy:
       matrix:
         ruby-version:
-          - 2.4
-          - 2.5
-          - 2.6
+          - '2.5'
+          - '2.6'
         gemfile:
           - rails4.2
           - rails5.0
           - rails5.1
           - rails5.2
           - rails6.0
           - rails6.1
-        exclude:
-          - ruby-version: 2.4
-            gemfile: rails6.0
-          - ruby-version: 2.4
-            gemfile: rails6.1
     env:
       BUNDLE_GEMFILE: gemfiles/${{ matrix.gemfile }}.gemfile
     steps:
-      - uses: zendesk/checkout@v2
+      - uses: zendesk/checkout@v3
       - uses: zendesk/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby-version }}
           bundler-cache: true
       - run: bundle exec rake test
+
+  specs_successful:
+    name: Specs passing?
+    needs: specs
+    if: always()
+    runs-on: ubuntu-latest
+    steps:
+      - run: |
+          if ${{ needs.specs.result == 'success' }}
+          then
+            echo "All specs pass"
+          else
+            echo "Some specs failed"
+            false
+          fi

migration_tools.gemspec

@@ -7,6 +7,8 @@ Gem::Specification.new "migration_tools", "1.7.0" do |s|
   s.files = `git ls-files lib`.split("\n")
   s.license = "Apache-2.0"
 
+  s.required_ruby_version = '>= 2.5.0'
+
   s.add_runtime_dependency "activerecord", '>= 4.2.0', '< 6.2'
 
   s.add_development_dependency "rake"