-
Notifications
You must be signed in to change notification settings - Fork 7
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 continuous integration #32
base: master
Are you sure you want to change the base?
Conversation
I notice this is significantly more configuration than the default Github Actions rust config. I assume this is for including clippy? Not sure if we want clippy to be a CI failure anyway |
For contrast, another project uses this config:
|
I believe we should include clippy, no harm doing so. This was taken from a template from rust github actions. |
@pickfire You mean https://github.com/actions-rs/ right? I'm talking about the default Github Actions stuff |
Yes |
I totally forgot - we added bors to this org, @bb010g what's the plan with that? |
Wow, we have bors? Nice. r? @expectocode |
r? @expectocode |
@pickfire i don't think it's set up |
Fix #23