Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clearly describe rm mv #105

Closed
wants to merge 2 commits into from
Closed

Clearly describe rm mv #105

wants to merge 2 commits into from

Conversation

brntbeer
Copy link
Member

@ZvikaZ, i wanted to bring this to your attention since it relates to the issue you opened (thank you by the way!)

@schacon you were last to edit this section, so I think you may be best to mention? What do you think of this addition? I know that in a nutshell section is basically repeating verbatim what was in the mv section, and I understand that rm and mv are essentially the same, but I think it could be misleading. Thoughts on accepting this?

closes #104

@brntbeer
Copy link
Member Author

brntbeer commented Dec 1, 2015

Ping @ZvikaZ and @schacon. Would love a 👍 from someone 😄

@brntbeer
Copy link
Member Author

I know people from @git/team-gitscm aren't exactly team-gitref, but would anyone mind giving this the 👍 it needs?

Essentially I'm just moving the aside section to be before git mv because it is only referencing git rm. The in a nutshell section then is able to sum up both pieces, even though it originally wasnt intended too, I feel it's more clear (see #104 for more motivation)

@brntbeer
Copy link
Member Author

🤷‍♂️

@brntbeer brntbeer closed this Jun 11, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'rm' and 'mv' description is mixed
1 participant