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

[pull] master from clap-rs:master #22

Open
wants to merge 6,170 commits into
base: master
Choose a base branch
from
Open

Conversation

pull[bot]
Copy link

@pull pull bot commented Oct 30, 2019

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label Oct 30, 2019
@pull pull bot added the merge-conflict Resolve conflicts manually label Feb 19, 2020
epage and others added 27 commits September 4, 2024 14:03
fix(help): Style arg groups
fix(complete): Put clap in chage of Fish ordering
Support `allow_hyphen_values` in native completions
No idea why using passing along `IFS` isn't workong on my machine (but is for tests).

Fixes #5730
fix(complete): Adjust how IFS is passed to clap
docs(complete): Clarify CompleteEnv's Shell trait
fix(complete): Strip wrappers for running completer
therealprof and others added 30 commits January 9, 2025 18:54
Turns out to be another win in case the suggestions feature is enabled

Signed-off-by: Daniel Egger <[email protected]>
…sertion-sort

Replace another sort_by call by a manual insertion with a binary_search
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
feat(mangen): Separate options based on `help_heading`
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Inspired by https://www.reddit.com/r/rust/comments/1i5np88/clap_documentation_is_too_confusing_for_me/m87mutm/

> > I always use derive, and I mostly use the reference. From manual
> experimentation I’ve had to develop the mental model that short
> implicitly is what makes an argument non-positional, and if I want an
> argument to be positional then it will be by default unless I specify
> short. When I search for the word “positional” in the reference, there
> are no results. When I search for it in the tutorial, there is one
> result that says something I can do with an argument that i’ve made
> positional, but there’s no indication for how to make the argument
> positional in the first place. The reference documentation on short
> doesn’t mention anything about the fact that that’s what determines
> whether an argument is positional, even when I click through to the
> builder documentation on Arg::short.
docs: Clarify the default state of an Arg/field
With the new-ish rustdoc header sidebar, hopefully this will make this
layout easiest to deal with.

By having everything on one page, people are less likely to miss
something, especially if they use their browsers search.
docs(tutorial): Experiment with a flat layout
fix(complete): Fix single quote escaping in PowerShell
fix(complete): Check if help string is empty
You can remove the env variable but sometimes you want to override it.
feat(complete): Provide a way to disable env completions
chore(deps): Update Rust crate clap-cargo to 0.15.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⤵️ pull merge-conflict Resolve conflicts manually
Projects
None yet
Development

Successfully merging this pull request may close these issues.