-
-
Notifications
You must be signed in to change notification settings - Fork 99
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 a warning if default-members are configured and no package selection arguments are passed in #695
Comments
!!!! That's a super serious issue. I checked out your branch at commit 7f51e7b5188d8b95d60ebbf9cdb8880b7da569b1 and I don't seem to be able to reproduce this, though:
(I had to press Ctrl-C because This is with cargo-nextest 0.9.45 on Linux x86_64. What version of cargo-nextest are you using and on what platform? |
I did another test run and this is what I got:
|
I also tried with https://gist.github.com/sunshowers/9149ee5b7c113ae364b68b56430cbe48 Maybe it's just that you were missing the |
Maybe For reference, here's a PR where I've been going through and fixing then: moonrepo/moon#492 |
Looks like If possible, it may be nice to display a warning if Cargo.toml is using workspaces but |
Yeah, you're the third person in the last 2 weeks to run into this. I'm going to repurpose this as a feature request to add that warning.
As a general rule, |
Good to know. Thanks for the quick response 👍 |
I started writing #701 but quickly realized it's a lot of work enumerating all the different cases. If you or someone else would like to do it, please comment here or over there! |
I've noticed something very weird recently. When running with
cargo nextest run
, the entire suite would pass with no issue:While debugging individual tests, I started noticing tests that were "passing" started failing, but after looking into them, they were definitely broken and had to be fixed. I have no idea why nextest was passing. Were this just being ignored/filtered? Was nextest failing to capture/run them? No idea.
When running with
cargo test --workspace
, there are many failing tests. I'm unable to run the entire suite to determine an exact number without fixing them all (which I'm in the process of doing anyways). You can verify all of this in my branch here: https://github.com/moonrepo/moon/tree/develop-0.21Things I've noticed that are failing:
#[serial]
fromserial_test
.#[should_panic]
.The text was updated successfully, but these errors were encountered: