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

Added commit graph #40

Merged
merged 2 commits into from
Apr 8, 2014

Conversation

linzhp
Copy link
Contributor

@linzhp linzhp commented Jun 11, 2013

Hi CVSAnalY team,

I decided to put my new changes to CVSAnalY instead of MininGit. This is my first batch of changes. It adds a commit graph to keep track of parents of commits. This commit graph currently only supports Git, as it is not necessary in Subversion and CVS. Hope someone could implement a Bzr version if it is necessary (I am not familiar with Bzr).

With regards to #1, I will pick features that I am actively using from MininGit and merge them into CVSAnalY incrementally, instead of merging all changes at once.

@ghost ghost assigned jgbarah Jun 12, 2013
@jgbarah
Copy link
Member

jgbarah commented Jun 12, 2013

Thanks a lot! This is something very much needed. The code seems ok, but I have to have a look at its interactions with the incremental retrieval of information from the git repo. I hope that won't take long time.

@andygrunwald
Copy link
Contributor

@jgbarah any new information here?

@andygrunwald
Copy link
Contributor

@linzhp Can you rebase this PR?
Then i will it give a try to test :)

@linzhp
Copy link
Contributor Author

linzhp commented Mar 23, 2014

Rebased

@andygrunwald
Copy link
Contributor

Hey @linzhp,

thanks for this awesome contribution.
I`ve checked this (with executing it with + without your changes + checking the commit_graph table). It works fine. Thanks :)

andygrunwald added a commit that referenced this pull request Apr 8, 2014
@andygrunwald andygrunwald merged commit 97a605d into MetricsGrimoire:master Apr 8, 2014
@linzhp linzhp deleted the feature/commit_graph branch April 9, 2014 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants