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

Clarify in --help that cargo test --all-targets excludes doctests #12422

Merged
merged 1 commit into from
Jul 31, 2023

Commits on Jul 31, 2023

  1. Mention that cargo test --all-targets excludes doctests

    Per my proposal here: rust-lang#6669 (comment)
    
    I tried to keep the edit minimalistic to match the surrounding style.
    
    If the maintainers are amenable to it, I think it could also be useful to do one or more of:
    - Offer concrete guidance on what to do to run actually-all tests (`--all-targets` then separately `--doc`).
    - Link to the issue at: rust-lang#6669
    - Mention that `cargo test` without `--all-targets` runs doctests by default, which seems not immediately obvious.
    
    I'd be happy to attempt to add any of the above that the maintainers feel would be a good fit here.
    obi1kenobi committed Jul 31, 2023
    Configuration menu
    Copy the full SHA
    2f59b20 View commit details
    Browse the repository at this point in the history