Skip to content
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

fix: Make cargo check to pass all tests #865

Merged
merged 1 commit into from
May 25, 2023
Merged

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented May 25, 2023

This command was failing:

cargo check  --benches --examples --tests --lib --bins --workspace

I think it would be easier to just use the cargo -p <crate> syntax in CI, instead of requiring cd ...?

This command was failing:

```bash
cargo check  --benches --examples --tests --lib --bins --workspace
```
@nyurik nyurik changed the title Make cargo check to pass all tests fix: Make cargo check to pass all tests May 25, 2023
Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

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

Thanks a lot (for all these improvements)!

I am trying to figure out what's happening with CI here as it now seems to fail consistently on one test, which unfortunately doesn't reproduce locally. Once I fixed that, I will happily merge this PR.

@Byron Byron merged commit 3a45973 into GitoxideLabs:main May 25, 2023
@nyurik nyurik deleted the fix-tests branch May 25, 2023 16:42
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.

2 participants