-
Notifications
You must be signed in to change notification settings - Fork 44
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
feat: Add limit support to allCommits query #856
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #856 +/- ##
========================================
Coverage 59.63% 59.64%
========================================
Files 155 155
Lines 17284 17294 +10
========================================
+ Hits 10308 10315 +7
- Misses 6045 6047 +2
- Partials 931 932 +1
|
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.
Maybe another test for order + limit?
I couldn't see how that could break here (note this PR only had to add to the parser and gql types, all order/mapper/limit code remains unchanged) |
736e477
to
6ee8f27
Compare
Added one with offset too in #859 as it makes the offset testing more robust IMO |
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.
LGTM
Didn't properly link a issue 😂 |
:D my bad, will close it the old fashioned way lol |
The good ol' days |
Relevant issue(s)
Resolves #
Description
Adds limit support to allCommits query. Let me know if you dont want it in 0.3.1.
Specify the platform(s) on which this was tested: