Skip to content

Conversation

@Therzok
Copy link
Member

@Therzok Therzok commented Jun 2, 2014

This closes #737.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be Rename(branch, newName, allowOverwrite)?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@nulltoken
Copy link
Member

@Therzok Neat! Just a nitpick. It looks like there's still some test code that invokes Branches.Move() in BranchFixture.cs

@Therzok
Copy link
Member Author

Therzok commented Jun 2, 2014

Oh? I must've missed it. One second.

@nulltoken
Copy link
Member

Last request. Could you please also rename the tests (eg. CanMove -> CanRename, ...)?

@Therzok
Copy link
Member Author

Therzok commented Jun 2, 2014

Oh, sure. :D

@Therzok
Copy link
Member Author

Therzok commented Jun 2, 2014

Everything should be done now. :o

@nulltoken nulltoken merged commit 8384e7e into libgit2:vNext Jun 2, 2014
@nulltoken
Copy link
Member

...and merged! THANKS! 😄

@nulltoken nulltoken added this to the v0.18.0 milestone Jun 2, 2014
@Therzok Therzok deleted the obsoleteMove branch June 2, 2014 15:27
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