Skip to content

clustering unit test - #348

Merged
mfeurer merged 2 commits into
developfrom
clust
Oct 12, 2017
Merged

clustering unit test#348
mfeurer merged 2 commits into
developfrom
clust

Conversation

@toon-vc

Copy link
Copy Markdown
Contributor

My second attempt to add the clustering unit test. If this is successful, I will close the other pull request and remove the 'clustering' branch.

@codecov-io

codecov-io commented Oct 11, 2017

Copy link
Copy Markdown

Codecov Report

Merging #348 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## develop #348 +/- ##
========================================
Coverage 89.69% 89.69% ========================================
Files 32 32 Lines 2522 2522 ========================================
Hits 2262 2262 Misses 260 260

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9664a0f...da5bb80. Read the comment docs.

@mfeurermfeurer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but you may want to add a pipeline unit test, too?

@mfeurer
mfeurer merged commit bd3a12b into developOct 12, 2017
@mfeurer
mfeurer deleted the clust branch October 12, 2017 08:38
Sign up for freeto 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.

3 participants

@toon-vc@codecov-io@mfeurer