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 GitHub Actions #230

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Conversation

TimWolla
Copy link
Contributor

Description

This updates the versions of the GitHub Actions to be compatible with node.js 20, avoiding a warning in CI runs.

I would recommend setting up Dependabot for GitHub Actions, which will automatically handle these updates going forward.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock
Copy link
Member

dblock commented Oct 21, 2024

I would recommend setting up Dependabot for GitHub Actions, which will automatically handle these updates going forward.

Dependabot is actually enabled on the repo, not sure what's up with it.

@dblock
Copy link
Member

dblock commented Oct 21, 2024

Rebase on top of #231?

This fixes:

> The following actions uses node12 which is deprecated and will be forced to
> run on node16: actions/checkout@v2. For more info:
> https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Signed-off-by: Tim Düsterhus <tim@tideways-gmbh.com>
Signed-off-by: Tim Düsterhus <tim@tideways-gmbh.com>
@TimWolla
Copy link
Contributor Author

Rebased both PRs.

@shyim shyim merged commit 3da6b1d into opensearch-project:main Oct 22, 2024
47 checks passed
@shyim
Copy link
Collaborator

shyim commented Oct 22, 2024

Thanks! :)

@TimWolla TimWolla deleted the ci-action-versions branch October 22, 2024 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants