Ed Page

> the result does not match what is documented. Could you point to the documentation in question? > default Cargo.toml doesn't have a description field Note that the examples do...

> > > I believe I've gone through the steps a new user would do while reading the first page of the docs and the result does not match what...

If we're in agreement that the demo should be idiomatic, then I'm not seeing what there is to change and would be inclined to close this issue.

So I feel we've come to a common understanding, even if not agreement, on several points but I worry this is going to be spinning in circles, so let's summarize...

`demo.md` **is** a test. We use the `trycmd` crate to execute the `console` blocks in our `md` files. That is why tests failed when #6099 tried to change `demo.md` only....

I would lean against that as it opens the question for explaining other parts which is the point of the docs we then link to after the example

In general, validation with global is a bit off, see #1546, particularly https://github.com/clap-rs/clap/issues/1546#issuecomment-1198809765 I'll keep this open to make sure we test and fix this case along with the others

The reason we limit the `Option`s is then a user typing `None` will have an ambiguous type. We could potentially add a convenience method on `Resettable`. I have been questioning...

`Command::defer` was primarily created for derives though most work to use it is deferred to clap v5. The proposed solution here (and written out in #6180) doesn't seem like it...

I'll have to think on this more. > I can remove the chaining. I just thought that it's more intuitive that replacing the existing function silently. Clap has a mixture...