-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
git-var: add page #5095
git-var: add page #5095
Conversation
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.
Thanks for the new page! I've left a comment below for you to review.
Co-authored-by: Starbeamrainbowlabs <[email protected]>
@bl-ue PR looks good but I have a question: in which context those commands that you have opened PRs for are used? It looks like most of the functionality can be achieved by using "git" and its subcommands. |
Co-authored-by: Axel Navarro <[email protected]>
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.
Looks ok to me after @navarroaxel's comment is addressed
@schneiderl I didn't see your comment!
Yes, I know that, and with commands like Do you agree? |
@bl-ue Agreed. I don't think we are the ones who decide which commands our user will use. Our job is to have everything documented as good as possible. 😄 |
common/
,linux/
, etc.)For #3953