Merge pull request #52 from insales/less_gems_in_ci

Do not install lint tools in all appraisals and remove unnecessary steps

.github/workflows/test.yaml

@@ -27,14 +27,15 @@ jobs:
 
     env:
       BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/rails_${{ matrix.rails-version }}.gemfile
+      BUNDLE_WITHOUT: lint
 
     services:
       postgres:
         image: postgres:12-alpine
         env:
+          POSTGRES_DB: paperclip_test
           POSTGRES_USER: postgres
           POSTGRES_PASSWORD: ""
-          POSTGRES_DB: postgres
           POSTGRES_HOST_AUTH_METHOD: trust
         ports:
           - 5432:5432
@@ -46,16 +47,7 @@ jobs:
           ruby-version: ${{ matrix.ruby-version }}
           bundler-cache: true
 
-      - name: Create databases
-        env:
-          RAILS_ENV: test
-          PGHOST: localhost
-          PGUSER: postgres
-          PGPORT: 5432
-        run: sudo -u postgres createdb -h localhost -p 5432 -U postgres paperclip_test
-
       - name: Run tests
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
-          bundle exec appraisal rails_${{ matrix.rails-version }} rake test
+          echo Running with $BUNDLE_GEMFILE
+          bundle exec rake test

Gemfile

@@ -24,11 +24,13 @@ gem 'thoughtbot-shoulda', '>= 2.9.0'
 gem 'pry'
 gem 'pry-byebug'
 
-gem 'rubocop', '0.81.0'
-gem 'rubocop-rails', '2.5.2'
-gem 'rubocop-rspec', '1.38.1'
-gem 'rubocop-performance', '1.5.2'
+group :lint do
+  gem 'rubocop', '0.81.0'
+  gem 'rubocop-rails', '2.5.2'
+  gem 'rubocop-rspec', '1.38.1'
+  gem 'rubocop-performance', '1.5.2'
 
-gem 'pronto', '>= 0.11', require: false
-gem 'pronto-brakeman', require: false
-gem 'pronto-rubocop', require: false
+  gem 'pronto', '>= 0.11', require: false
+  gem 'pronto-brakeman', require: false
+  gem 'pronto-rubocop', require: false
+end

gemfiles/rails_52.gemfile

@@ -15,12 +15,15 @@ gem "mocha"
 gem "thoughtbot-shoulda", ">= 2.9.0"
 gem "pry"
 gem "pry-byebug"
-gem "rubocop", "0.81.0"
-gem "rubocop-rails", "2.5.2"
-gem "rubocop-rspec", "1.38.1"
-gem "rubocop-performance", "1.5.2"
-gem "pronto", ">= 0.11", require: false
-gem "pronto-brakeman", require: false
-gem "pronto-rubocop", require: false
+
+group :lint do
+  gem "rubocop", "0.81.0"
+  gem "rubocop-rails", "2.5.2"
+  gem "rubocop-rspec", "1.38.1"
+  gem "rubocop-performance", "1.5.2"
+  gem "pronto", ">= 0.11", require: false
+  gem "pronto-brakeman", require: false
+  gem "pronto-rubocop", require: false
+end
 
 gemspec path: "../"

gemfiles/rails_60.gemfile

@@ -15,12 +15,15 @@ gem "mocha"
 gem "thoughtbot-shoulda", ">= 2.9.0"
 gem "pry"
 gem "pry-byebug"
-gem "rubocop", "0.81.0"
-gem "rubocop-rails", "2.5.2"
-gem "rubocop-rspec", "1.38.1"
-gem "rubocop-performance", "1.5.2"
-gem "pronto", ">= 0.11", require: false
-gem "pronto-brakeman", require: false
-gem "pronto-rubocop", require: false
+
+group :lint do
+  gem "rubocop", "0.81.0"
+  gem "rubocop-rails", "2.5.2"
+  gem "rubocop-rspec", "1.38.1"
+  gem "rubocop-performance", "1.5.2"
+  gem "pronto", ">= 0.11", require: false
+  gem "pronto-brakeman", require: false
+  gem "pronto-rubocop", require: false
+end
 
 gemspec path: "../"

gemfiles/rails_61.gemfile

@@ -15,12 +15,15 @@ gem "mocha"
 gem "thoughtbot-shoulda", ">= 2.9.0"
 gem "pry"
 gem "pry-byebug"
-gem "rubocop", "0.81.0"
-gem "rubocop-rails", "2.5.2"
-gem "rubocop-rspec", "1.38.1"
-gem "rubocop-performance", "1.5.2"
-gem "pronto", ">= 0.11", require: false
-gem "pronto-brakeman", require: false
-gem "pronto-rubocop", require: false
+
+group :lint do
+  gem "rubocop", "0.81.0"
+  gem "rubocop-rails", "2.5.2"
+  gem "rubocop-rspec", "1.38.1"
+  gem "rubocop-performance", "1.5.2"
+  gem "pronto", ">= 0.11", require: false
+  gem "pronto-brakeman", require: false
+  gem "pronto-rubocop", require: false
+end
 
 gemspec path: "../"

gemfiles/rails_70.gemfile

@@ -15,12 +15,15 @@ gem "mocha"
 gem "thoughtbot-shoulda", ">= 2.9.0"
 gem "pry"
 gem "pry-byebug"
-gem "rubocop", "0.81.0"
-gem "rubocop-rails", "2.5.2"
-gem "rubocop-rspec", "1.38.1"
-gem "rubocop-performance", "1.5.2"
-gem "pronto", ">= 0.11", require: false
-gem "pronto-brakeman", require: false
-gem "pronto-rubocop", require: false
+
+group :lint do
+  gem "rubocop", "0.81.0"
+  gem "rubocop-rails", "2.5.2"
+  gem "rubocop-rspec", "1.38.1"
+  gem "rubocop-performance", "1.5.2"
+  gem "pronto", ">= 0.11", require: false
+  gem "pronto-brakeman", require: false
+  gem "pronto-rubocop", require: false
+end
 
 gemspec path: "../"