- Notifications
You must be signed in to change notification settings - Fork302
JSONAPI prefix for paginators#463
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
codecov-io commentedAug 21, 2018 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## master #463 +/- ##==========================================+ Coverage 93.33% 93.37% +0.03%========================================== Files 56 56 Lines 3164 3183 +19 ==========================================+ Hits 2953 2972 +19 Misses 211 211
Continue to review full report at Codecov.
|
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. Thanks.
…for consistency (django-json-api#463)"This reverts commite6290af.
Description of the Change
Per adiscussion with@sliverc, renamed recently-added pagination classes from
JsonApi
-prefix toJSONAPI
-prefix. This includes new deprecation warnings for the old names.I also added a minor fix to tox.ini not to run flake8 on the entire directory but only the relevant files.
You can cherry-pick that out if you prefer it not to be part of this PR.
Checklist
CHANGELOG.md
AUTHORS
(This replaces#458 where I messed up the history by rebasing incorrectly.)