-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update CI #18
base: main
Are you sure you want to change the base?
Update CI #18
Conversation
Should probably fix clippy errors first, to see the actual CI runs. |
- Run all CI on ubuntu (no platform-specific code) - Switch to codecov - Update MIRIFLAGS Remove: - valgrind/fuzzing (stubs) - rust-code-analysis and - weighted-code-coverage
- Remove anyhow::Error field from Error variant - Rename/move to solver::NoiseModelStatus
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Looks like it still expects the old CI checks before auto-merge is allowed. That might be a github repo setting? |
@lu-zero I am not sure who has the permissions for this, but can you add a codecov token for this repository (instructions)? Or maybe ping someone who can? Thanks in advance |
Closes #15.
Prerequisite:
CODECOV_TOKEN
to this repositoryChanges:
Removed:
rust-code-analysis
andweighted-code-coverage
could arguably be added back, but I'm not sure it's necessary to run these in CI.