-
Notifications
You must be signed in to change notification settings - Fork 268
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 #392
Conversation
517fed2
to
fee3614
Compare
Ok, loom test failure will be fixed by Xudong-Huang/generator-rs#20, and everything else passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent!
Hmm..., it seems that we need to change the "required statuses" of the branch protection settings to merge this (requires admin rights). @seanmonstar Could you change the branch protection settings? (I guess you have admin rights) |
@taiki-e I think they're all set (the list has a lot of old options to require). |
@seanmonstar Thanks! I confirmed they are set. I'm going to merge this... |
No description provided.