Skip to content

Add GitHub Actions CI #1146

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

Merged
merged 2 commits into from
Dec 4, 2020
Merged

Add GitHub Actions CI #1146

merged 2 commits into from
Dec 4, 2020

Conversation

Xanewok
Copy link
Collaborator

@Xanewok Xanewok commented Dec 2, 2020

Builds on #1144 (now merged) so that the tests pass.

Fixes #1145

The spin-up time is very fast and a single build runs 3-5 minutes (all 4 builds run in parallel), and so that's an improvement over what we have: AppVeyor runs 5-7 mins, Travis runs 15-20 mins (but needs some time to spin up). This also adds testing for macOS.

See https://github.com/Xanewok/racer/actions/runs/396511445 for a successful run.

r? @kngwyu

@Xanewok
Copy link
Collaborator Author

Xanewok commented Dec 2, 2020

I believe the action will be only triggered and run once it's committed to stop potentially malicious actors from extracting CI secrets and so on via PRs.

Copy link
Collaborator

@kngwyu kngwyu left a comment

Choose a reason for hiding this comment

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

Thanks!
Could please also remove .appveyor.yml?
Since now we run tests for Windows on actions, it is no longer useful.

- master
- '*'
schedule:
- cron: '0 0 * * *' # Nightly at 00:00 UTC
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice 👍 (as our build is often broken 😅

@Xanewok
Copy link
Collaborator Author

Xanewok commented Dec 4, 2020

Could please also remove .appveyor.yml?

Done.
Do you want me to remove .travis.yml as well, if we're removing old CI setup here?

@kngwyu
Copy link
Collaborator

kngwyu commented Dec 4, 2020

Do you want me to remove .travis.yml as well, if we're removing old CI setup here?

Yeah, but let's remove it after watching the actions work. Thanks!

@kngwyu kngwyu merged commit 3cf8109 into racer-rust:master Dec 4, 2020
@Xanewok Xanewok deleted the github-actions branch December 4, 2020 15:08
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.

Consider switching CI to GitHub Actions
2 participants