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

Upgrade github workflow upload-artifact@v3 to @v4. #294

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

fishybow
Copy link
Contributor

upload-artifact@v3 will be deprecate in Jan 2025. Find more details here: https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md

@fishybow
Copy link
Contributor Author

As a side benefit, the OTA zip can now be downloaded individually.

@fishybow
Copy link
Contributor Author

The only slight annoyance is that the all-in-one merged archive may be a bit harder to locate among all the other files.

It's named OLED/LCD__pixljs_fw_* now. One possible improvement is rename to something like OLED/LCD__pixljs_fw_final_download_*. But I will leave the renaming to you, @solosky.

@solosky
Copy link
Owner

solosky commented Dec 5, 2024

Hi there,

Thank your PR on the workflow, I already completed the workflow update on neo branch, I may cherry pick to the develop branch when I am avaliable.

@solosky
Copy link
Owner

solosky commented Dec 5, 2024

Meanwhile, app like iNFC and MToolsBLE already adopted current build artifact layout, we would better do not change the final artifact layout, at least make it capatible.

…ge make sure the user will be backed out of the screen when that happens, instead of being permanently stuck on the naming screen.
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