feat(completer): Added completion for --features flag
#15309
Conversation
rustbot
added
A-cli
|
@epage suggestion will be appreciated on how can i improve this like should i include some description ? or something like that, as i have not discussed with yu earlier and pick directly from the parent issue |
src/cargo/util/command_prelude.rs
Outdated
| Ok(features | ||
| .into_iter() | ||
| .map(|name| clap_complete::CompletionCandidate::new(name)) | ||
| .collect()) |
There was a problem hiding this comment.
Should we sort or de-duplicate?
There was a problem hiding this comment.
i have sorted it in b615137
There was a problem hiding this comment.
So we'll show a feature multiple times if the name is used in different packages? In some workspaces, that could really bloat the output.
There was a problem hiding this comment.
However, we can always iterate on this in a future PR.
There was a problem hiding this comment.
So we'll show a feature multiple times if the name is used in different packages? In some workspaces, that could really bloat the output.
ahh, we should have a check ig if the package already exists in the completer, that sounds good ?
There was a problem hiding this comment.
I would assume that we put all features into a BTree<String, Vec<String>> that maps feature names to packages. We then generate the candidates from this so you get one feature name no matter how many packages use it.
There was a problem hiding this comment.
should i implement in this pr ? or in the feature iteration ?
There was a problem hiding this comment.
My assumption would be this PR.
Is there a reason to push it out?
There was a problem hiding this comment.
However, we can always iterate on this in a future PR.
i thought you wanted to iterate on future pr
Gmin2
force-pushed
the
completer-features
branch
from
9674bef to
b615137
Compare
This comment has been minimized.
This comment has been minimized.
|
FYI I'm fine with you cleaning up your commits through the review process. This is small and easy to track the differences between each posting. |
Gmin2
force-pushed
the
completer-features
branch
from
a0315e2 to
1f18cdf
Compare
|
Please clean up the commits for how they should be merged. |
Gmin2
force-pushed
the
completer-features
branch
from
1fbc09a to
64b6a59
Compare
- Added completion for --features flag - Collect features from all workspaces - TODO: To remove a feature multiple times if the name is used in different packages
Gmin2
force-pushed
the
completer-features
branch
from
64b6a59 to
c5d4950
Compare
What does this PR try to resolve?
This attempts to complete the autocompleter for
cargo build --features <TAB>,cargo run --features <TAB>It loads all the features that are there in the profile section of Cargo.toml
Related to #14520
How should we test and review this PR?
by running
cargo build --features <TAB>,cargo run --features <TAB>