Hi Ryan,

I had a look at the code and it looks excellent, so I think I should
merge it into the branch.

One minor thing is that it seems that the Go community uses the
convention that all docstrings should start with the entity being
documented. See, for example,
http://golang.org/pkg/regexp/#Regexp.ReplaceAllFunc.

It's a minor thing, but I am quite new to the Go community and I usually
try to follow the conventions that are already in place. It avoids a lot
of problems down the road.

Best wishes,
Mats Kindahl

I'll merge it as On 12/26/2013 06:29 AM, RyanMarcus wrote:
>
> In directed/storage.go:
>
> > @@ -142,7 +142,7 @@ func (graph *Graph) HasEdge(source, target Vertex) bool {
> >  	return false
> >  }
> >  
> > -// Order will return the order of the graph, that isnumber of vertices
> > +// Order will return the order of the graph, that is, the number of vertices
>
> This is just a typo correction. Hadn't even realized I committed it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mkindahl/gograph/pull/8/files#r8552971>.
>