8000 submodules.asc: clarify DbConnector/try-merge example by salewski · Pull Request #1420 · progit/progit2 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

submodules.asc: clarify DbConnector/try-merge example #1420

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

Merged
merged 1 commit into from

Apr 21, 2020

Conversation

salewski
Copy link
Contributor
@salewski salewski commented Apr 21, 2020

Fix AsciiDoc source,console block to render separately the consecutive commands git branch try-merge c771610 and git merge try-merge. This is accomplished by separating the commands with an empty line, and by omitting the unneeded "(DbConnector) " prefix in front of the git merge... command prompt. Puts the example into the form consistent with the other command blocks, and also enhances readability.

State explicitly in the prose that "try-merge" will be the name of the branch.

Fix AsciiDoc 'source,console' block to render separately the consecutive
commands 'git branch try-merge c771610' and 'git merge try-merge'. This is
accomplished by separating the commands with an empty line, and by omitting
the unneeded "(DbConnector) " prefix in front of the 'git merge...' command
prompt. Puts the example into the form consistent with the other command
blocks, and also enhances readability.

State explicitly in the prose that "try-merge" will be the name of the branch.
@salewski salewski force-pushed the ads/clarify-try-merge-in-ch7-submodules branch from 31f877d to 6629642 Compare April 21, 2020 19:11
@ben
Copy link
Member
ben commented Apr 21, 2020

Oh yeah, that's probably some leftover copy-paste junk. Thanks for improving this!

@ben ben merged commit 812855b into progit:master Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0