-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 FAQ section to site #1095
Add FAQ section to site #1095
Conversation
This is just a start, and there are some more items that we will probably want to add. The import section probably is really a start to typelevel#912, and may end up simply referencing a separate import document, as discussed in typelevel#936. But I thought it would be nice to get something out there as an interim resolution.
Current coverage is 88.88%@@ master #1095 diff @@
==========================================
Files 226 226
Lines 2916 2916
Methods 2865 2865
Messages 0 0
Branches 48 48
==========================================
Hits 2592 2592
Misses 324 324
Partials 0 0
|
o.orEmpty | ||
``` | ||
|
||
**Note**: if you import `cats.implicits._` (the preferred method), you should _not_ also use imports like `cats.std.syntax._` or `cats.std.either._`. This can result in ambiguous implicit values that cause bewildering compile errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect you meant something different than cats.std.syntax._
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops quite right thanks. I wish I could have doobie verify inline code! I'll fix this but I might not have time today.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
This is a great start. Let's merge it and iterate! 👍 |
Invoking the Documentation Rule and merging :) |
This is just a start, and there are some more items that we will
probably want to add.
The import section probably is really a start to #912, and may end up
simply referencing a separate import document, as discussed in #936. But
I thought it would be nice to get something out there as an interim
resolution.