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

ci: update github actions #213

Merged
merged 3 commits into from
Dec 18, 2023
Merged

ci: update github actions #213

merged 3 commits into from
Dec 18, 2023

Conversation

vbarua
Copy link
Member

@vbarua vbarua commented Dec 18, 2023

No description provided.

@@ -78,18 +78,18 @@ jobs:
run: |
value=`mv isthmus/build/graal/isthmus isthmus/build/graal/isthmus-${{ matrix.os }}`
- name: Publish artifact
uses: actions/upload-artifact@master
uses: actions/upload-artifact@v4
Copy link
Member Author

@vbarua vbarua Dec 18, 2023

Choose a reason for hiding this comment

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

I suspect that this is what was causing the release failure. All of the download actions are @v3, but the upload action is @master. The upload-artifact action had a 4.0 release recently, and I suspect the @v3 version of download-artifact can't handle the v4 uploads.

@danepitkin
Copy link
Contributor

LGTM!

@vbarua vbarua merged commit 45ecf85 into main Dec 18, 2023
8 checks passed
@vbarua vbarua deleted the vbarua/ci-actions-updates branch December 18, 2023 19:19
ajegou pushed a commit to ajegou/substrait-java that referenced this pull request Mar 29, 2024
* ci: upgrade to checkout@v4
* ci: upgrade to upload and download artifact @v4
* ci: upgrade to setup-node@v4
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.

2 participants