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 Windows Release Builds #114
Conversation
bc9cce5
to
41f2956
Compare
41f2956
to
36c9c16
Compare
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.
I haven't manually tested these due to them being build-and-release changes, but all looks good. Only left a few nitpicks.
npm run make | ||
If ($LastExitCode -ne 0) { Exit $LastExitCode } | ||
|
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.
npm run make | |
If ($LastExitCode -ne 0) { Exit $LastExitCode } | |
npm run make | |
If ($LastExitCode -ne 0) { Exit $LastExitCode } |
npm run make | ||
If ($LastExitCode -ne 0) { Exit $LastExitCode } | ||
|
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.
npm run make | |
If ($LastExitCode -ne 0) { Exit $LastExitCode } | |
npm run make | |
If ($LastExitCode -ne 0) { Exit $LastExitCode } |
install_gems | ||
bundle exec fastlane distribute_release_build | ||
agents: | ||
queue: "mac" # |
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.
queue: "mac" # | |
queue: "mac" |
const releaseVersionZipFilename_mac = `https://cdn.a8c-ci.services/studio/studio-darwin-v${ version }.app.zip`; | ||
const releaseVersionZipFilename_x64 = `https://cdn.a8c-ci.services/studio/studio-darwin-x64-v${ version }.app.zip`; | ||
const releaseVersionZipFilename_arm64 = `https://cdn.a8c-ci.services/studio/studio-darwin-arm64-v${ version }.app.zip`; | ||
const releaseVersionZipFilename_win32 = `https://cdn.a8c-ci.services/studio/studio-win32-v${ version }.exe`; |
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.
Interesting mix of camel- and snake- casing.
Did you use it to better visually distinguish the platform?
Did you consider:
const releaseVersionZipFilename_mac = `https://cdn.a8c-ci.services/studio/studio-darwin-v${ version }.app.zip`; | |
const releaseVersionZipFilename_x64 = `https://cdn.a8c-ci.services/studio/studio-darwin-x64-v${ version }.app.zip`; | |
const releaseVersionZipFilename_arm64 = `https://cdn.a8c-ci.services/studio/studio-darwin-arm64-v${ version }.app.zip`; | |
const releaseVersionZipFilename_win32 = `https://cdn.a8c-ci.services/studio/studio-win32-v${ version }.exe`; | |
const releaseVersionZipFilenameMac = `https://cdn.a8c-ci.services/studio/studio-darwin-v${ version }.app.zip`; | |
const releaseVersionZipFilenameX64 = `https://cdn.a8c-ci.services/studio/studio-darwin-x64-v${ version }.app.zip`; | |
const releaseVersionZipFilenameArm64 = `https://cdn.a8c-ci.services/studio/studio-darwin-arm64-v${ version }.app.zip`; | |
const releaseVersionZipFilenameWin32 = `https://cdn.a8c-ci.services/studio/studio-win32-v${ version }.exe`; |
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.
Addressed in ed8c467
* Extract logic to distribute builds in Fastlane * DRY builds folder definition in Fastfile * Remove leftover debug `puts` * DRY S3 bucket name * Fix a comment's wording * DRY further * DRY even more * DRY commit, build, etc in Fastfile
Proposed Changes
Creates signed Windows builds each time a new tag is created.
Testing Instructions
See https://a8c.slack.com/archives/C06DRMD6VPZ/p1715362399654439, note that each build is downloadable (and signed)