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 initial GitHub actions #230

Merged
merged 1 commit into from
Sep 21, 2020
Merged

Add initial GitHub actions #230

merged 1 commit into from
Sep 21, 2020

Conversation

wilzbach
Copy link
Member

@s-ludwig
Copy link
Member

Hm, is it normal that some of the jobs are still pending after hours?

@wilzbach wilzbach force-pushed the github-actions branch 2 times, most recently from f64f0d0 to dc51391 Compare September 20, 2020 06:51
@wilzbach
Copy link
Member Author

Hm, is it normal that some of the jobs are still pending after hours?

Not really. Though, it seems to have worked flawless on the rerun I triggered.

I have now added all jobs that are run on AppVeyor, s.t. we could already remove AppVeyor.
We probably need to see how good the through-put is before replacing Travis as there's a lot of long-running jobs there.

@wilzbach
Copy link
Member Author

We'll need to update the README shield (see e.g. badges/shields#2574 (comment)), but I think this is only available once it is in the main branch.

@s-ludwig
Copy link
Member

It says 3.x minutes, which would be awesome, although AppVeyor also did a release build. I have disabled the AppVeyor check now and made the GitHub ones required instead.

@s-ludwig s-ludwig merged commit 4357b32 into master Sep 21, 2020
@s-ludwig s-ludwig deleted the github-actions branch September 21, 2020 08:52
kubo39 added a commit to kubo39/vibe-core that referenced this pull request Sep 27, 2021
AppVeyor was replaced by GitHub Actions in vibe-d#230.
@kubo39 kubo39 mentioned this pull request Sep 27, 2021
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.

2 participants