Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add syntax test for |@| on FreeApplicative #570

Merged
merged 1 commit into from
Oct 15, 2015

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Oct 15, 2015

This is a test that is fixed by #568. I just thought it might be nice to have a regression test for the specific case that was discovered to not work before that PR.

Big thanks to @stew for the fix and @smungee for reporting the issue.

@codecov-io
Copy link

Current coverage is 75.68%

Merging #570 into master will not affect coverage as of 5e131b9

@@            master    #570   diff @@
======================================
  Files          157     157       
  Stmts         2163    2163       
  Branches        68      68       
  Methods          0       0       
======================================
  Hit           1637    1637       
  Partial          0       0       
  Missed         526     526       

Review entire Coverage Diff as of 5e131b9

Powered by Codecov. Updated on successful CI builds.

@non
Copy link
Contributor

non commented Oct 15, 2015

👍

1 similar comment
@fthomas
Copy link
Member

fthomas commented Oct 15, 2015

👍

fthomas added a commit that referenced this pull request Oct 15, 2015
Add syntax test for |@| on FreeApplicative
@fthomas fthomas merged commit cb804ce into typelevel:master Oct 15, 2015
@ceedubs ceedubs deleted the freeapp-cinnabon branch October 15, 2015 12:19
Sign up for free to 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.

4 participants