Skip to content
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

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
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.

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