]> git.proxmox.com Git - cargo.git/commit
Auto merge of #11075 - epage:feat, r=weihanglo
authorbors <bors@rust-lang.org>
Tue, 13 Sep 2022 17:08:01 +0000 (17:08 +0000)
committerbors <bors@rust-lang.org>
Tue, 13 Sep 2022 17:08:01 +0000 (17:08 +0000)
commitc633b27cb8db3e3919a0f8622a38f135b9a27cf3
treed4e0ceaeb230c8de2cdf05edcba51166e1cfa2fd
parentfedd172f3a47a750ed0c91c98860e048a8049a4c
parent5a6cfc9a56f822dc982b6c5c3c961d9ac3ba3f82
Auto merge of #11075 - epage:feat, r=weihanglo

fix(add): Clarify which version the features are added for

### What does this PR try to resolve?

This gives a hint to users that we might not be showing the feature list
for the latest version but the earliest version.

Also when using a workspace dependency, this is a good reminder of what the version requirement is that was selected.  That could also be useful for reused dependencies but didn't want to bother with the relevant plumbing for that.

ie we are going from
```console
$ cargo add chrono@0.4
    Updating crates.io index
      Adding chrono v0.4 to dependencies.
             Features:
             - rustc-serialize
             - serde
```
to
```console
$ cargo add chrono@0.4
    Updating crates.io index
      Adding chrono v0.4 to dependencies.
             Features as of v0.4.2:
             - rustc-serialize
             - serde
```

### How should we test and review this PR?

I'd recommend looking at this commit-by-commit.  This is broken up into several refactors leading up the main change.  The refactors are focused on pulling UI logic out of dependency editing so we can more easily evolve the UI without breaking the editing API.  I then tweaked the behavior in the final commit to be less redundant / noisy.

The existing tests are used to demonstrate this is working.

### Additional information

I'm also mixed on whether the meta version should show up.

Fixes #11073