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 actions/checkout to v4 #520

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Update actions/checkout to v4 #520

merged 1 commit into from
Sep 6, 2023

Conversation

Uaitt
Copy link
Contributor

@Uaitt Uaitt commented Sep 5, 2023

On the 4th of September 2023 actions/checkout released its version 4.

With this PR I bump all the occurrences of actions/checkout@v3 in the CI to its newer version. Additionally, I have also updated the README.md to reflect the current changes (and I have removed a few trailing whitespaces too).

For more information about the new version of actions/checkout take a look at their release page: https://github.com/actions/checkout/releases/tag/v4.0.0

README.md Outdated
@@ -191,7 +191,7 @@ They should be set in the `env` at the job level as shown in the [example](#matr
To find the correct the environment variable name, see the [Bundler docs](https://bundler.io/man/bundle-config.1.html) or look at `.bundle/config` after running `bundle config --local KEY VALUE` locally.
You might need to `"`-quote the environment variable name in YAML if it has unusual characters like `/`.

To perform caching, this action will use `bundle config --local path $PWD/vendor/bundle`.
Copy link
Member

Choose a reason for hiding this comment

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

Could you undo such changes? The trailing spaces are intentional to force a line break when rendering Markdown.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, fixed!

I am not sure why I thought removing them was a good idea tho 😅

Thanks for pointing that out 😃

@Uaitt Uaitt requested a review from eregon September 6, 2023 06:04
@eregon eregon merged commit 4f1c6a7 into ruby:master Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants