Skip to content

Conversation

braedon
Copy link
Contributor

@braedon braedon commented Aug 8, 2017

The actual maximums for these params vary by plan/subscription, so to
support all don't try and validate against a limit.

The actual maximums for these params vary by plan/subscription, so to
support all don't try and validate against a limit.
@braedon braedon requested a review from robertkern August 8, 2017 01:56
@coveralls
Copy link

coveralls commented Aug 8, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling b3a3ccb on no-offset-limit into a356101 on master.

Copy link
Member

@robertkern robertkern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@braedon braedon merged commit 93d1a1a into master Aug 8, 2017
@braedon braedon deleted the no-offset-limit branch August 8, 2017 02:03
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.

3 participants