Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add new shapes to Unapply #568

Merged
merged 1 commit into from
Oct 15, 2015
Merged

Add new shapes to Unapply #568

merged 1 commit into from
Oct 15, 2015

Conversation

stew
Copy link
Contributor

@stew stew commented Oct 15, 2015

This commit adds Unapply shapes which will match a datum with a type in the
shape F[X[_], Y] with a typeclass expecting a F[_]

Thanks to @smungee who noticed that apply syntax was not working with
FreeApplicative because of this.

This commit adds Unapply shapes which will match a datum with a type in the
shape F[X[_], Y] with a typeclass expecting a F[_]

Thanks to @smungee who noticed that apply syntax was not working with
FreeApplicative because of this.
@codecov-io
Copy link

Current coverage is 75.68%

Merging #568 into master will decrease coverage by -0.05% as of 41f264c

@@            master    #568   diff @@
======================================
  Files          157     157       
  Stmts         2155    2159     +4
  Branches        68      68       
  Methods          0       0       
======================================
+ Hit           1632    1634     +2
  Partial          0       0       
- Missed         523     525     +2

Review entire Coverage Diff as of 41f264c

Powered by Codecov. Updated on successful CI builds.

@adelbertc
Copy link
Contributor

👍

1 similar comment
@non
Copy link
Contributor

non commented Oct 15, 2015

👍

non added a commit that referenced this pull request Oct 15, 2015
@non non merged commit 757ff73 into typelevel:master Oct 15, 2015
@non non removed the in progress label Oct 15, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants