-
Notifications
You must be signed in to change notification settings - Fork 254
chore: housekeeping #11
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
Conversation
|
Thanks for the PR! Want to add that clippy command to the workflow as well? |
yes, so it's converted as draft, wiil submitt later along with other changes. |
|
Test / coverage (pull_request) fail with error
|
|
Not familiar with CodeCov! Looking through it now. Also apologies got a few conflicts from merging another PR. Overall love the action here. Really want to get this in |
@tekaratzas better get this pr merged if you got time, thanks |
|
Hey apologies for the delay. Not super keen on getting the codecov key tbh. Feel it may be overkill and don't want to add a bunch of API dependencies at this point. Would you be ok removing it for now? Otherwise this looks great. |
|
Amazing, thanks for sticking with this! |




cargo clippy --workspace --all-targets --all-features -- -D warnings