Page MenuHomePhabricator

[CI] Build macOS and Windows desktop app on github
ClosedPublic

Authored by michal on Jan 3 2023, 7:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 12:44 AM
Unknown Object (File)
Sun, Apr 21, 12:44 AM
Unknown Object (File)
Sun, Apr 21, 12:44 AM
Unknown Object (File)
Sun, Apr 21, 12:43 AM
Unknown Object (File)
Sun, Apr 21, 12:41 AM
Unknown Object (File)
Apr 2 2024, 12:36 AM
Unknown Object (File)
Apr 2 2024, 12:36 AM
Unknown Object (File)
Apr 2 2024, 12:36 AM
Subscribers

Details

Summary

ENG-2532
Adds GitHub workflows for building the desktop app on Windows and macOS. The Windows one should be especially useful because nobody develops on a Windows machine so if it breaks we wouldn't catch it until later.
They are heavily inspired by the Android/ iOS workflows.

Test Plan

Tested on a fork but without secrets for now.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

.github/workflows/macos_ci.yml
24–44

Taken from the iOS workflow

.github/workflows/windows_ci.yml
29–35

The general idea was taken from the Android workflow but I'm using a Windows utility certutil for decoding base64.

michal requested review of this revision.Jan 3 2023, 7:37 AM

Looks good, let's remove the pull_request trigger though

.github/workflows/macos_ci.yml
12–13

We actually don't want these to run on pull request. Secrets aren't available by default for PRs because it's possible for someone malicious to modify workflow and put up a PR that eg writes secrets to a file and then uploads them or something along those lines.

I'll put up a diff right now to cut them from the other workflows

This revision is now accepted and ready to land.Jan 3 2023, 10:48 AM

I'll put up a diff right now to cut them from the other workflows

https://phab.comm.dev/D6150

Remove pull_request trigger