-
-
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-cherry-pick: add page #1196
Conversation
Thanks! Would it make sense to add another example, say, to cherry pick into a branch that isn't the current one? |
I don't believe you can do that with cherry-pick. I think you need to checkout the branch you want to cherry-pick into. Should I add something about that to it? |
This reverts commit 0f404f3.
Yeah, the "to the current branch" should probably be mentioned in the main description, if that's always the case. Perhaps adapting one or two of the examples from https://git-scm.com/docs/git-cherry-pick#_examples would be a better, then. But it's just a suggestion to improve the page, nothing that would block merging :) |
I've updated it to specify the current branch in the main description |
👍 |
@alefir unfortunately the latest change doesn't conform to the the current page format guidelines. @agnivade any idea why the linter isn't detecting that? |
@waldyrious Would you prefer removing that line, moving it to the description, or something else? |
@waldyrious - oops, it actually fails but somehow the exit code is 0. Is it due to the tee or the error redirection ? https://travis-ci.org/tldr-pages/tldr/builds/184122860. Wonder how it did not get caught in our tests. EDIT: yes, its due to |
Hopefully Travis will now report errors correctly. One of them, I believe, should be that the main description lines should end with a period :) |
Added page for git cherry-pick
EDIT: Fixes #1155