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

Fix tut in monad doc #1445

Merged
merged 1 commit into from
Oct 30, 2016
Merged

Fix tut in monad doc #1445

merged 1 commit into from
Oct 30, 2016

Conversation

guersam
Copy link
Contributor

@guersam guersam commented Oct 28, 2016

Not sure of the underlying machinary, but it turned out that the missing closing brace has been causing mysterious tut passing found on #1444.

@codecov-io
Copy link

codecov-io commented Oct 28, 2016

Current coverage is 92.19% (diff: 100%)

Merging #1445 into master will not change coverage

@@             master      #1445   diff @@
==========================================
  Files           242        242          
  Lines          3615       3615          
  Methods        3543       3543          
  Messages          0          0          
  Branches         72         72          
==========================================
  Hits           3333       3333          
  Misses          282        282          
  Partials          0          0          

Powered by Codecov. Last update 91242d0...acbe892

@adelbertc
Copy link
Contributor

adelbertc commented Oct 28, 2016

I've noticed this in my own tut files from time to time.. hopefully this is the last of them. @tpolecat do you know what might cause the old file to still pass tut?

Might be lack of closing brace makes tut think there's still input and if the file just doesn't close tut is fine with that?

@tpolecat
Copy link
Member

@adelbertc yeah Cody discovered that a while ago and I haven't fixed it yet, obviously. tpolecat/tut#65

@adelbertc
Copy link
Contributor

👍

@adelbertc adelbertc merged commit bcc751f into typelevel:master Oct 30, 2016
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