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.

Sign up for GitHub

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

Jump to bottom

Improve development experience #410

Merged
merged 5 commits into from Oct 9, 2023
Merged

Improve development experience #410

merged 5 commits into from Oct 9, 2023

Commits on Oct 9, 2023

  1. cargo-insta: remove MetadataCommand reexport

    @tamird
    tamird committed Oct 9, 2023
    Configuration menu
    Copy the full SHA
    ab666f3 View commit details
    Browse the repository at this point in the history

  2. integration-tests: str::strip_suffix

    @tamird
    tamird committed Oct 9, 2023
    Configuration menu
    Copy the full SHA
    c09de8c View commit details
    Browse the repository at this point in the history

  3. integration-tests: assert cargo-insta success

    @tamird
    tamird committed Oct 9, 2023
    Configuration menu
    Copy the full SHA
    652fb5f View commit details
    Browse the repository at this point in the history

  4. integration-tests: assert cargo-insta output 

    Clarify that `cargo insta test` is expected to produce a change in the
snaphots.
    @tamird
    tamird committed Oct 9, 2023
    Configuration menu
    Copy the full SHA
    dc08881 View commit details
    Browse the repository at this point in the history

  5. Cargo.toml: add crates to workspace 

    This allows IDE integration to work when the root of the repository is
opened. Prior to this change, a developer would have to open cargo-insta
to get code completion for that crate.

This also causes clippy to run on all crates, so this commit fixes a few
lints.

The integration-tests crate's test is changed to a proper test function
so that it can rely on always being run from the root of the
integration-tests crate. Care is taken to remove the tests directory
after running to avoid pollution in which `cargo test` attempts to run
those tests.
    @tamird
    tamird committed Oct 9, 2023
    Configuration menu
    Copy the full SHA
    8bb9ea8 View commit details
    Browse the repository at this point in the history