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

Generate electron-updater metadata #1397

Conversation

rafaellehmkuhl
Copy link
Member

@rafaellehmkuhl rafaellehmkuhl commented Oct 11, 2024

This PR is needed before #1396. It creates a new standard for the binary artifact filenames and generates the latest.yml files needed for the update system.

I made sure everything is right on the PR artifacts, but after we merge it we should check if the release artifacts are also right. The release is also working fine.

@ArturoManzoli
Copy link
Contributor

ArturoManzoli commented Oct 11, 2024

It did build normally on my local, but didn't deploy. It's asking for some version on package.json. Am I missing something?
image

I see it deployed normally on the GitHub CI tho.
image

@rafaellehmkuhl
Copy link
Member Author

It did build normally on my local, but didn't deploy. It's asking for some version on package.json. Am I missing something?

This happens because you don't have any tags in your local git repository. After you create one it will work as expected.

@rafaellehmkuhl rafaellehmkuhl marked this pull request as draft October 14, 2024 13:17
@rafaellehmkuhl
Copy link
Member Author

Converting back to draft as I noticed the .zip file is not being generated for macOS releases, which prevents auto-updates to work for them.

@rafaellehmkuhl rafaellehmkuhl force-pushed the generate-electron-update-metadata branch 3 times, most recently from 816f90d to b6ed394 Compare October 14, 2024 14:29
@rafaellehmkuhl rafaellehmkuhl marked this pull request as ready for review October 14, 2024 14:53
@rafaellehmkuhl rafaellehmkuhl force-pushed the generate-electron-update-metadata branch from b6ed394 to 1457a87 Compare October 14, 2024 14:53
@rafaellehmkuhl
Copy link
Member Author

Fixed. Ready for review.

The new version fixes the usage of `c.extraMetadata` on Windows
The `latest.yml` files are now generated, pointing to the artifact binaries, with the metadata needed for the `electron-updater` system to work.
@rafaellehmkuhl rafaellehmkuhl force-pushed the generate-electron-update-metadata branch from 1457a87 to 6b81bc3 Compare October 14, 2024 16:50
@ArturoManzoli ArturoManzoli merged commit 5e01d7b into bluerobotics:master Oct 15, 2024
10 checks passed
@rafaellehmkuhl rafaellehmkuhl deleted the generate-electron-update-metadata branch October 15, 2024 10:50
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