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

Migrate CI to GitHub Actions #6476

Merged
merged 41 commits into from
Aug 3, 2022
Merged

Conversation

DomClark
Copy link
Member

Every time some issue comes up with CI, moving to GitHub Actions is brought up. Well, here we go.

@JohannesLorenz JohannesLorenz self-requested a review July 31, 2022 12:52
@PhysSong PhysSong self-requested a review July 31, 2022 13:31
Copy link
Contributor

@JohannesLorenz JohannesLorenz left a comment

Choose a reason for hiding this comment

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

Only 2 minor comments.

Main issues:

  • You're using artifacts, but in your action tab, I can't find out how to download them -> I was too dumb to scroll down 😁
  • The LMMS bot did not run for this PR. Can we somehow verify that it's still working?

Copy link
Contributor

@Veratil Veratil left a comment

Choose a reason for hiding this comment

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

Just a few notes, nothing blocking.

DomClark added 5 commits July 31, 2022 18:31
The host version doesn't matter since we're using Docker, so use
something we won't have to update manually.
Ninja should be able to determine this automatically.
@DomClark
Copy link
Member Author

The LMMS bot did not run for this PR. Can we somehow verify that it's still working?

It only works for CircleCI and AppVeyor builds, and I removed the configuration files for those in this PR. In order to download artifacts from GitHub Actions, you need to be logged into GitHub, so posting raw links to artifacts won't work. However there are third-party services that can provide download links, such as nightly.link. The bot could be changed to link to that site, or as a temporary replacement, https://github.com/oprypin/nightly.link/blob/master/.github/workflows/pr-comment.yml might be worth trying. Eventually I think it would be nice to provide nightly and PR downloads via lmms.io, but that's getting more and more out of scope.

@DomClark DomClark merged commit fe91a65 into LMMS:master Aug 3, 2022
@DomClark DomClark deleted the github-actions-build branch August 3, 2022 16:29
@lukas-w
Copy link
Member

lukas-w commented Aug 3, 2022

I've been receiving random emails about failed builds from AppVeyor & CircleCI for years now with no way to turn them off. Finally those days are over. Thank you @DomClark 🙏

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.

5 participants