Skip to content

Commit

Permalink
Merge pull request #54 from kickstarter/use-env-var-in-test-not-publish
Browse files Browse the repository at this point in the history
use `matrix.activerecord` in `test` not publish, drop Ruby 2.7 from CI
  • Loading branch information
a-lavis authored Oct 15, 2024
2 parents aa619e7 + adea25f commit 5b7778f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/test-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ jobs:
strategy:
matrix:
ruby:
- "2.7"
- "3.1"
- "3.2"
- "3.3"
Expand All @@ -33,7 +32,7 @@ jobs:
with:
ruby-version: ${{ matrix.ruby }}
- run: docker compose up --detach
- run: bundle install --jobs 4 --retry 3
- run: ACTIVE_RECORD_VERSION=${{ matrix.activerecord }} bundle install --jobs 4 --retry 3
- run: bundle exec rake bootstrap
- run: bundle exec rake spec
push:
Expand All @@ -57,5 +56,5 @@ jobs:
:rubygems_api_key: ${{ secrets.RUBYGEMS_API_KEY }}
YAML
chmod 0600 ~/.gem/credentials
- run: ACTIVE_RECORD_VERSION=${{ matrix.activerecord }} bundle install
- run: bundle install
- run: bundle exec rake release

0 comments on commit 5b7778f

Please sign in to comment.