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

Fix graph display issues #638

Merged
merged 9 commits into from
Jul 4, 2017
Merged

Fix graph display issues #638

merged 9 commits into from
Jul 4, 2017

Conversation

swegener
Copy link
Contributor

@swegener swegener commented Jul 2, 2017

These commits fix a couple of graph display issues. Most issues have been found by running the graph drawing code on a large repo. The added regression tests are initially committed in wrong state and then fixed up to make it easier to visualize the issues they fix.

It also adds two new drawing types for merge commits that are also used as the parent for several other commits.

@jonas
Copy link
Owner

jonas commented Jul 4, 2017

It looks like this fixes #419 and supersedes #605.

@jonas jonas merged commit 440d03d into jonas:master Jul 4, 2017
@jonas
Copy link
Owner

jonas commented Jul 4, 2017

Thank you @swegener. Awesome to see the graph rendering improved.

@jonas jonas modified the milestone: tig-2.3 Jul 8, 2017
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