-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add building and signing for all architectures #1615
Conversation
resources/latest-mac.yml
Outdated
version: VERSION | ||
files: | ||
- url: Realm Studio-VERSION-mac.zip | ||
sha512: ZIP_SHA | ||
size: ZIP_SIZE | ||
blockMapSize: ZIP_BLOCKMAP_SIZE | ||
- url: Realm Studio-VERSION.dmg | ||
sha512: DMG_SHA | ||
size: DMG_SIZE | ||
path: Realm Studio-VERSION.zip | ||
sha512: ZIP_SHA | ||
RELEASE_DATE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this committed on purpose? If so, what is it used for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is referenced in the commented block for creating auto-update files for mac in the scenario where electron builder can't be relied on to correctly write that out - not used right now, but I thought it's a good idea to keep that documented if it ever comes up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that case, I'd prefer if this was put into a markdown file with an introduction 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably best would be sticking it into the comment for that code block, with annotation where it needs to go if that is needed again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thank you for documenting it with many comments. Should we update docs/RELEASING.md
?
83b4d4e
to
c6414ff
Compare
c6414ff
to
5583844
Compare
This adds building, signing and release uploading for all architectures, and prepares adding universal builds for macOS as well as ARM builds for Windows/Linux.
Missing is the slack notification, and the changelog generation - that is mostly copy/paste from other projects, but we probably need to discuss if we should stick that in this workflow as well, or do a separate workflow for that.