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

Build changes #152

Merged
merged 4 commits into from
Jul 26, 2022
Merged

Build changes #152

merged 4 commits into from
Jul 26, 2022

Conversation

adamboutcher
Copy link
Collaborator

This should enable Mac M1 Builds and fix some issues with the Windows build not working correctly.
We do however lose Mac 32bit builds but I doubt they're needed.

@adamboutcher adamboutcher added the Build build-related changes label Jul 26, 2022
@adamboutcher
Copy link
Collaborator Author

@jemand771 Can you double check I've not done anything stupid please?

Copy link
Member

@jemand771 jemand771 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see anything that's obviously wrong, but the only way to know for sure is to test it :b

@jemand771
Copy link
Member

Note that github uses the workflow files from the default branch (dev, which you're merging into) for all branches - not the one on the current branch

A way to test this would be putting the workflow file on dev in a fork and trying the merge there, but I don't reaally have time for that right now :c

@adamboutcher
Copy link
Collaborator Author

I tested it on my own fork and it worked, I just wanted to make sure it looked sane to someone else incase I broke something obvious to others.

Thank you for your review.

@adamboutcher adamboutcher merged commit 6ba6401 into statping-ng:dev Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build build-related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants