remove confusing outdated note on interactive rebasing #3460
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
xabbuh
commented
Jan 10, 2014
xabbuh
commented
| Q | A |
|---|---|
| Doc fix? | yes |
| New docs? | no |
| Applies to | all |
| Fixed tickets | #3458 |
xabbuh
referenced
this pull request
in xabbuh/symfony-docs
Jan 10, 2014
|
|
||
| .. note:: | ||
|
|
||
| ``-f`` is a shortcut for the ``--force`` switch. |
There was a problem hiding this comment.
we not use --force in the docs to make it all clear?
There was a problem hiding this comment.
No, but the note above used --force before which is quite confusing, isn't it?
There was a problem hiding this comment.
If we use --force everywhere in this article instead of -f, there is no confusing and we don't need such a note.
There was a problem hiding this comment.
+1 for using long options; explaining Git's short options should not be part of Symfony's docs.
There was a problem hiding this comment.
Good idea. Replaced -f with --force
|
Thanks for taking care of this @xabbuh - very awesomely handled |
weaverryan
added a commit
that referenced
this pull request
Jan 21, 2014
…buh) This PR was merged into the 2.3 branch. Discussion ---------- remove confusing outdated note on interactive rebasing | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | all | Fixed tickets | #3458 Commits ------- eec82ed remove confusing outdated note on interactive rebasing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.