Skip to content

Adds additional tests for Option T#692

Merged
non merged 1 commit into
typelevel:masterfrom
mikejcurry:option-t-tests
Nov 24, 2015
Merged

Adds additional tests for Option T#692
non merged 1 commit into
typelevel:masterfrom
mikejcurry:option-t-tests

Conversation

@mikejcurry

@mikejcurry mikejcurry commented Nov 21, 2015

Copy link
Copy Markdown
Contributor
  • Tests consistency between flatMap and flatMapF
  • Tests consistency between Show[OptionT] instance and OptionT.show

@mikejcurry
Adds additional tests for Option T
9714421 
* Tests consistency between flatMap and flatMapF
* Tests consistency between Show[OptionT] instance and OptionT.show
@mikejcurry mikejcurry mentioned this pull request Nov 21, 2015
Closed
@codecov-io

codecov-io commented Nov 21, 2015

Copy link
Copy Markdown

Current coverage is 84.12%

Merging #692 into master will increase coverage by +0.23% as of 88e4119

@@            master    #692   diff @@
======================================
  Files          162     162       
  Stmts         2217    2217       
  Branches        74      74       
  Methods          0       0       
======================================
+ Hit           1860    1865     +5
  Partial          0       0       
+ Missed         357     352     -5

Review entire Coverage Diff as of 88e4119

Powered by Codecov. Updated on successful CI builds.

@ceedubs

ceedubs commented Nov 22, 2015

Copy link
Copy Markdown
Contributor

👍 thanks!

@non

non commented Nov 24, 2015

Copy link
Copy Markdown
Contributor

👍 great thanks!

non added a commit that referenced this pull request Nov 24, 2015
@non
Merge pull request #692 from mikejcurry/option-t-tests
b50dd05 
Adds additional tests for Option T
@non non merged commit b50dd05 into typelevel:master Nov 24, 2015
@mikejcurry mikejcurry deleted the option-t-tests branch December 11, 2015 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mikejcurry @codecov-io @ceedubs @non