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

Move Graphviz to own repo? #97

Closed
clemens-tolboom opened this issue Jan 6, 2014 · 5 comments · Fixed by #115
Closed

Move Graphviz to own repo? #97

clemens-tolboom opened this issue Jan 6, 2014 · 5 comments · Fixed by #115
Assignees
Milestone

Comments

@clemens-tolboom
Copy link
Collaborator

In several issues we run into particular graphviz related issues. These should probably not belong to graph repo perse.

What we could do is move this to graph-uml but that library still tries to become graphviz independent.

Maybe graphviz repo is ok? https://packagist.org/search/?q=graphviz has some hits so maybe not the best name.

@clue
Copy link
Member

clue commented Jan 8, 2014

Indeed, this issue has been raised several times already.

I'm not opposed against this at all. Unless anybody can find a reason to not split this off into a separate repository, I'll add this to my TODOs.

The other graphviz packages have a somewhat different focus, so it makes sense to keep this as a separate repository which is associated to this graph library as its core.

@clue
Copy link
Member

clue commented Jan 30, 2014

Graphviz will eventually be split off to graphp/graphviz, see also the linked ticket graphp/graphviz#1.

@clue
Copy link
Member

clue commented Mar 22, 2014

Also GraphViz is the only component that takes advantage of this Layoutable class, so it will be removed from this repository as well. Instead, ticket #100 will eventually add a general purpose collection of properties, which can also include GraphViz layout attributes, so eventually we can drop the whole Layoutable class.

Because of this, this ticket is blocked until #100 is addressed. Any input is welcome.

@clue
Copy link
Member

clue commented Dec 31, 2014

Ticket #100 has been resolved via #103, we can now split this off.

@clue clue removed the blocked label Dec 31, 2014
@clue
Copy link
Member

clue commented Dec 31, 2014

The new repository depends on #103, so we should tag a v0.8.0 release before splitting this off. As such, this ticket will be moved to the v0.9.0 milestone.

NVM, GraphViz will be removed via #115, the separate package currently targets this master branch and will be updated immediately after to target the v0.8.0 release.

@clue clue modified the milestones: v0.9.0, v0.8.0 Dec 31, 2014
@clue clue modified the milestones: v0.8.0, v0.9.0 Dec 31, 2014
@clue clue closed this as completed in #115 Dec 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants