Skip to content

Fix: Update artifact upload action to v4 #47

Merged
beesaferoot merged 2 commits into
mainfrom
ci-switch-artifact-version
May 2, 2025
Merged

Fix: Update artifact upload action to v4 #47
beesaferoot merged 2 commits into
mainfrom
ci-switch-artifact-version

Conversation

@beesaferoot
Copy link
Copy Markdown
Collaborator

Brief summary of the change made

Older versions are now deprecated. See https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Are there any other side effects of this change that we should be aware of?

Describe how you tested your changes?

Pull Request checklist

Please confirm you have completed any of the necessary steps below.

  • Meaningful Pull Request title and description
  • Changes tested as described above
  • Added appropriate documentation for the change.
  • Created GitHub issues for any relevant followup/future enhancements if appropriate.

Comment thread .github/workflows/deploy-gh-pages.yaml Outdated

- name: Upload artifact
uses: actions/upload-pages-artifact@v1
uses: actions/upload-artifact@v4
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this works for Pages deployment. From what I remember you need to use

actions/upload-pages-artifact@v3
actions/deploy-pages@v4

and update them together.

See

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh I see, Thanks for pointing out 🙌🏾

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Updated.

@beesaferoot beesaferoot requested a review from dmohns May 1, 2025 13:48
@beesaferoot beesaferoot merged commit 4e18660 into main May 2, 2025
8 checks passed
@beesaferoot beesaferoot deleted the ci-switch-artifact-version branch May 2, 2025 09:11
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