Conversation
Current coverage is 94.73%@@ master #9 diff @@
==========================================
Files 1 2 +1
Lines 21 57 +36
Methods 0 0
Messages 0 0
Branches 1 4 +3
==========================================
+ Hits 20 54 +34
Misses 0 0
- Partials 1 3 +2
|
[manifold.deferred :as md])) | ||
|
||
(defn paginated->stream | ||
"Given a resource with pagination semantics, turns it into a single |
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 think this should read "Given a resource with pagination semantics, turn it into a single stream that ..."
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.
What about "Given a resource with pagination semantics, returns a single stream that ..."
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 think @derwolfe's objection is about person, not the word itself; so that'd be "return", I guess.
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.
@lvh - you are correct about my objection.
LGTM assuming ci passes. |
an HTTP request). `get-results` is passed that value from `handle-todo`, and | ||
should return the results in that value. These will be added to the output | ||
stream. `get-next-todo` should return the next todo or nil if this is the | ||
last page. `first-todo` is the first todo item to kick this process off." |
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 was just about to ask you to document these parameters. Good stuff.
No description provided.