-
Notifications
You must be signed in to change notification settings - Fork 299
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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 commit e6290af.
Description of the Change
Per a discussion 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.mdAUTHORS(This replaces #458 where I messed up the history by rebasing incorrectly.)