Skip to content

Add testing on newer versions of DRF and Django #30

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

Merged
merged 10 commits into from
May 18, 2018

Conversation

kevin-brown
Copy link
Contributor

@kevin-brown kevin-brown commented May 17, 2018

This library hasn't changed much and still appears to be supported in newer versions of Django REST framework, but it's difficult to verify because the test matrix stops at DRF 3.1. This updates the test matrix for newer versions of DRF.

Changes:

  • Added testing for DRF versions up to DRF 3.8
  • Added testing for Django versions up to Django 2.0
  • Reorganized testing matrix to only test Python and Django versions that are supported by the DRF versions

This maintains testing and support for the existing DRF and Django combinations, since no code changes were necessary to support newer combinations.

This is the last DRF version to support Django 1.8, which is the
latest Django version which is included in the test matrix.
This adjusts Travis to match the testing matrix set up within Tox.
This adjusts the Tox requirements to use the `.*` format to pick up
the latest version in the range, so we don't need to worry about
pinning to a specific version and having a newer patch release come
out later down the line.
While these supports versions past DRF 3.6, this commit is only adding
matrix support up to 3.6 for now to minimize overall changes.
At this point the testing matrix is massive, somewhere in the range
of 80 different builds. This should probably get cut down at some
point, but that would likely warrant breaking changes or a stable
version support policy (which doesn't currently appear to exist).
These two versions only support Django 1.10 and Django 1.11, so
that makes testing a bit easier and reduces the number of new
matrix combinations.
This is only supported by two versions of Django REST framework,
since it's a relatively recent release.
Django 2.0 does not support Python 2.7 and Python 3.3, so it is not
tested on those combinations. It's only tested on Python 3.4.

We are now almost at 100 combinations on Travis...
@jpadilla
Copy link
Owner

@kevin-brown thank you so much for this ✨

@jpadilla jpadilla merged commit ab41878 into jpadilla:master May 18, 2018
@kevin-brown kevin-brown deleted the drf-versions branch May 19, 2018 03:19
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.

2 participants