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 ci.yml (to use non-deprecated upload-artifact action) #91

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

meghfossa
Copy link

This updates to v4, as v3 is deprecated (and will actively throw error in CI)
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Starting January 30th, 2025, GitHub Actions customers will no longer be able to use v3 of actions/upload-artifact or actions/download-artifact. Customers should update workflows to begin using v4 of the artifact actions as soon as possible. While v4 of the artifact actions improves upload and download speeds by up to 98% and includes several new features, there are key differences from previous versions that may require updates to your workflows. Please see the documentation in the project repositories for guidance on how to migrate your workflows.

@meghfossa meghfossa changed the title Update ci.yml Update ci.yml (to use non-deprecated upload-artifact action) Feb 1, 2025
@JohnBra
Copy link
Owner

JohnBra commented Feb 1, 2025

Hey @meghfossa, thanks for opening this PR!

I just checked the breaking changes of v4 and it lists the deprecation of uploading artifacts with the same name.

Currently the zip will keep the name vite-web-extension-chrome with every upload. Can you confirm running the updated workflow will not throw an error after running it multiply times?

If it does throw an error, I suggest appending the version from the manifest.json or package.json to the filename.

@meghfossa
Copy link
Author

I ran into following into CI -

Error: This request has been automatically failed because it uses a deprecated version of actions/upload-artifact: v3. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

I resolved it by bumping the version on my fork (so not a blocker or issue for me). Thanks for creating this template - this literally saved hours of js/ts/env setup. Feel free to close this PR

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