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

Beta: Fix config profiles using "dev" in cargo test. #8013

Merged
merged 1 commit into from
Mar 17, 2020

Commits on Mar 17, 2020

  1. Auto merge of rust-lang#8012 - ehuss:fix-config-profile-test, r=alexc…

    …richton
    
    Fix config profiles using "dev" in `cargo test`.
    
    Fix a bug where the "dev" profile was not loaded from config when running `cargo test` when "dev" is not listed in `Cargo.toml`.
    
    There was a mistake in rust-lang#7750 where it did not consider implicit profiles. Config profiles need to be loaded explicitly in order to properly handle environment variables. However, it was only looking at the profile requested on the command-line and those listed in `Cargo.toml`. `cargo test` also implicitly uses the "dev" profile for dependencies, so make sure those are loaded from config as well.
    bors authored and ehuss committed Mar 17, 2020
    Configuration menu
    Copy the full SHA
    a8fe057 View commit details
    Browse the repository at this point in the history