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

Add Github Action workflows #189

Merged
merged 7 commits into from
Jun 23, 2021
Merged

Add Github Action workflows #189

merged 7 commits into from
Jun 23, 2021

Conversation

networkfusion
Copy link
Contributor

@networkfusion networkfusion commented Jan 22, 2021

Adds github action workflow for a FREE CI.

It can probably be improved, but it is a good working starting point and significantly improves binary generation for Windows on check-ins and PR's (including on peoples forks).

Note: the first run will take significantly longer due to downloading packages. In subsequent runs they will be cached.

relates to: #160 and #132

@networkfusion
Copy link
Contributor Author

networkfusion commented Jan 22, 2021

As a note, the build output (which would be done in this repo instead once the the PR is approved) can be seen here: https://github.com/networkfusion/cen64/actions

@networkfusion networkfusion mentioned this pull request Jan 22, 2021
@mikeryan
Copy link
Member

Thank you, this looks awesome! I'm happy to merge it whenever you think it's ready. Do you have any more changes to push, or can I merge now?

@networkfusion
Copy link
Contributor Author

networkfusion commented Jan 25, 2021

@mikeryan Sorry for the delay, I didnt get a notification (due to me having the repo ignored?!). There are no more changes planned, as all seems to work as it should.
Though I am open to changes to the cmake flags used, I am no expert on them...

@networkfusion
Copy link
Contributor Author

@mikeryan did you receive my last comment???

@sp1187 sp1187 merged commit 5503dd0 into n64dev:master Jun 23, 2021
@networkfusion networkfusion deleted the github-actions branch July 9, 2021 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants