Update setup-ruby env in ci.yml - Githubissues

ixti / sidekiq-throttled

Concurrency and rate-limit throttling for Sidekiq

MIT License

709 stars 76 forks source link

Update setup-ruby env in ci.yml #106

Closed zrod closed 2 years ago

zrod commented 2 years ago

Hi,

I noticed the current Github CI workflow is pulling the latest Sidekiq gem regardless of the specified gemfile. This PR should prevent that.

More details, latest main (example):

ruby 2.7, sidekiq 5.0 https://github.com/sensortower/sidekiq-throttled/runs/4526523043?check_suite_focus=true

run ruby/setup-ruby@v1 step output:

143   Installing sidekiq 6.3.1

With PR changes:

ruby 2.7, sidekiq 5.0 https://github.com/zrod/sidekiq-throttled/runs/4538987147?check_suite_focus=true

run ruby/setup-ruby@v1 step output:

143   Installing sidekiq 5.0.5

More info: https://github.com/ruby/setup-ruby/#matrix-of-gemfiles

ixti commented 2 years ago

@zrod Thank you very much! Great catch!