Fixed #2 #5

Merged
merged 2 commits into from Jan 11, 2017

Projects

None yet

2 participants

@luccitan
Contributor

eslint has run and is ok.

index.js
@@ -21,7 +21,9 @@ function connectedComponents(graph) {
return [];
if (!graph.size)
- return graph.nodes();
+ return graph.nodes().map(function(node) {
@Yomguithereal
Yomguithereal Jan 11, 2017 Member

Avoid using map. => for loop

@luccitan
Contributor

Last review fixed with commit 2976ac6

@Yomguithereal
Member

This one is good to go. Merging. Thanks @luccitan.

@Yomguithereal Yomguithereal merged commit f91eba0 into graphology:master Jan 11, 2017
@luccitan luccitan deleted the unknown repository branch Jan 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment