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

change git log order,add status type "Rename". #53

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

change git log order,add status type "Rename". #53

wants to merge 3 commits into from

Conversation

cyqw
Copy link

@cyqw cyqw commented Sep 1, 2014

No description provided.

add a new status "RenameModify",

if the file not modified when rename, use "Rename".it modified, use "RenameModify"
self.commitDirs(t)
cc_exec(['mv', '-nc', self.old, self.new])

class RenameModify(Rename):
Copy link
Owner

Choose a reason for hiding this comment

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

Can we share more of the base class where possible? Like __init__ and the first part of commit?

@charleso
Copy link
Owner

charleso commented Sep 6, 2014

Sorry for the delay, happy to merge after both comments are addressed (I may be wrong on the second point)

@charleso
Copy link
Owner

charleso commented Sep 6, 2014

Also, over the years I've softened on my stance on comments. Feel free to add some if you like. :)

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.

3 participants