-
Notifications
You must be signed in to change notification settings - Fork 11
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
CI build using GitHub Actions #4
base: master
Are you sure you want to change the base?
Conversation
cf20f4e
to
5ccc4d1
Compare
Thank you so much for the binaries! I've been having difficulty deploying it. I will take a look in a bit! |
Okay, so I found an issue. I think you may need to include the libraries/frameworks for each platform's binary. It gives me an error because of this. |
5ccc4d1
to
96ff095
Compare
Build for Linux (Ubuntu), macOS and Windows Saves archive with built application per platform
96ff095
to
c9edb9a
Compare
Added bundling of QT dependencies on macOS and Windows. There's is a third party vendoring tool for Linux that can do the same there but I'd assume Linux users rather use their shared QT dependencies managed by the system https://github.com/Dahlgren/OnHubDesktop/actions/runs/212665064. |
Nice, thank you! And this will add build and other related checks to each merge, right? I'm new to GitHub Actions. |
Correct, it will run for all pushes and all pull requests |
The Windows client seem to crash for me but I don't have a Google WiFi setup to test with atm |
I will merge this when I get the chance to test it. Thank you! |
Build for Linux (Ubuntu), macOS and Windows
Saves archive with built application per platform