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
Extend cross referencing options with values #10883
Conversation
de04e51
to
f0abef2
Compare
What do you think @AA-Turner? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
8e83d4d
to
0cd9f58
Compare
This supports a commonly used format when it comes to optional argument: -fauto-profile[=path], where `path` is optional part. Plus it supports a format when space is used: -fauto-profile path. Extends: sphinx-doc#10840 Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
0cd9f58
to
861f1b7
Compare
@marxin please don't force push, it makes review much more difficult after leaving comments. A |
All right, so you prefer leaving there the "Suggested change" commits, right? |
Yes, I squash merge on commit. A |
Thanks! A |
Makes sense, will do that next time. |
I thank you for a nice negotiation we made. |
This supports a commonly used format when it comes to optional argument:
-fauto-profile[=path]
, wherepath
is optional part. Plus it supports a format when space is used:-fauto-profile path
.Extends: #10840