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 two small errors in OptionT docs #825

Merged
merged 1 commit into from
Jan 26, 2016
Merged

Conversation

rgcase
Copy link
Contributor

@rgcase rgcase commented Jan 25, 2016

Fixed a small error I noticed where Option[Future[String]] was supposed to say Future[Option[String]]. Also added a "]" where one was missing.

I'm really interested in this project so I'll probably be reading a lot of the documentation. I'll try to keep an eye out for things like this as I go.

@codecov-io
Copy link

Current coverage is 89.12%

Merging #825 into master will not affect coverage as of bd15bd3

@@            master    #825   diff @@
======================================
  Files          168     168       
  Stmts         2309    2309       
  Branches        75      75       
  Methods          0       0       
======================================
  Hit           2058    2058       
  Partial          0       0       
  Missed         251     251       

Review entire Coverage Diff as of bd15bd3

Powered by Codecov. Updated on successful CI builds.

@stew
Copy link
Contributor

stew commented Jan 25, 2016

👍

@ceedubs
Copy link
Contributor

ceedubs commented Jan 26, 2016

Wonderful. Thanks, @rgcase! 👍

ceedubs added a commit that referenced this pull request Jan 26, 2016
Fix two small errors in OptionT docs
@ceedubs ceedubs merged commit 6dc0c32 into typelevel:master Jan 26, 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