Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update GH action #121

Merged
merged 3 commits into from
Dec 3, 2021
Merged

Update GH action #121

merged 3 commits into from
Dec 3, 2021

Conversation

nachiket87
Copy link
Contributor

@nachiket87 nachiket87 commented Dec 2, 2021

Type of PR (feature, enhancement, bug fix, etc.)

Enhancement.

Description

actions/setup-ruby is deprecated and should be replaced with the maintained ruby/setup-ruby

Keep GH actions updated and remove deprecated dependencies.

Checklist

  • My code follows the style guidelines of this project
  • Checks (StandardRB & Prettier-Standard) are passing

actions/setup-ruby is deprecated and should be replaced with the maintained ruby/setup-ruby
@julianrubisch
Copy link
Contributor

Thanks a ton 👏

@julianrubisch
Copy link
Contributor

Ah, forgot that with setup-ruby you have to specify the complete ruby version... could you do that? I think 3.0.3 should be fine!

@rickychilcott
Copy link
Contributor

Using...

- uses: ruby/setup-ruby@v1
      with:
        ruby-version: 3.0.3
        bundler-cache: true

Could both install ruby 3 and cache gems. Thus reducing next action as well :)

@@ -19,12 +19,6 @@ jobs:
with:
ruby-version: 3.0.3
bundler-cache: true
- uses: actions/cache@v1
with:
path: vendor/bundle
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rickychilcott Is this okay?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems right to me! Thank you.

@julianrubisch julianrubisch merged commit 59ad36f into stimulusreflex:master Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants