Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

ApplyK #40

Merged
merged 3 commits into from
Jul 9, 2018
Merged

ApplyK #40

merged 3 commits into from
Jul 9, 2018

Conversation

LukaJCB
Copy link
Contributor

@LukaJCB LukaJCB commented Jul 7, 2018

Should fix #38

@codecov
Copy link

codecov bot commented Jul 7, 2018

Codecov Report

Merging #40 into master will decrease coverage by 6.35%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #40      +/-   ##
=========================================
- Coverage   93.85%   87.5%   -6.36%     
=========================================
  Files          18      21       +3     
  Lines         179     192      +13     
  Branches       14      14              
=========================================
  Hits          168     168              
- Misses         11      24      +13
Impacted Files Coverage Δ
.../scala/mainecoon/laws/discipline/ApplyKTests.scala 0% <0%> (ø)
...aws/src/main/scala/mainecoon/laws/ApplyKLaws.scala 0% <0%> (ø)
core/src/main/scala/mainecoon/ApplyK.scala 0% <0%> (ø)

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 a58150f...dc8883a. Read the comment docs.

@kailuowang
Copy link
Owner

Thanks! I'll add the annotation macro soon.

@kailuowang kailuowang merged commit 7d7434e into kailuowang:master Jul 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an ApplyK type class?
2 participants