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
Improve git issue help
output
#29
Conversation
Git's own help, for reference:
|
And
|
Very nice, thank you! One small problem: unfortunately you updated the wrong file. Look at the comment before your text:
Also, please verify the correctness of all other synced documents after your change. |
My fault for not seeing the comment. 😃 Will take significantly more time to do this right, but I'll give it a shot. |
No problem, thank you! |
Fixes in README: * Group commands logically and add headers * Rephrase some help texts for clarity Fixes in sync-docs.sh script: * Remove repetitive `git issue` * Remove repetitive colons * Remove repetitive fullstops * Align commands and explanations like `git help`
Updated and added a test that checks whether the files are in sync. Feedback is welcome 😄 PS. The files weren't in sync after your commit today! 😛 |
I implemented Will open a PR here when this PR is merged. |
fail "make sync-docs changed $GenFiles" | ||
git checkout -- $GenFiles | ||
fi | ||
|
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.
LGTM. Please note that running sync-docs should result in an empty git-status.
That's exactly what this test checks. 😄
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.
Excellent! Please ping me when ready to merge.
LGTM too. Ready to merge. |
Thank you! I admired your sed wizardry. Quite rare these days. |
Make it look like git's own help:
git issue
git help