Skip to content
This repository has been archived by the owner on May 26, 2020. It is now read-only.

Update of django, drf, python version handling in tox.ini #262

Merged
merged 3 commits into from
Mar 2, 2017

Conversation

angvp
Copy link
Contributor

@angvp angvp commented Sep 15, 2016

  • Removed tests for drf 2.4, 3.0 because django versions 1.8,1.9,1.10 doesn't play well with them.
  • Untie drf and django to specific releases for a version, and instead get the latest release of every version

Some of these changes were done in #252 but weren't merged because #256 was more complete adding support of drf3.4 and django 1.10, however I rather like to test against the latest release of a version so this is the main motivation of this PR.

Regards.

@angvp angvp force-pushed the update-tox branch 3 times, most recently from ab7d32c to 7306651 Compare September 15, 2016 14:08
so now is tied to the newest release of a specific version
@angvp
Copy link
Contributor Author

angvp commented Feb 28, 2017

Closing it because it got no interest from the maintainers.

Note to the maintainers: Please drop some feedback to ALL the PR that you have open, merge, reject them or whatever but give feedback, the standby status is the worst part.

@angvp angvp closed this Feb 28, 2017
@jpadilla jpadilla reopened this Feb 28, 2017
@jpadilla
Copy link
Owner

@angvp definitely interested, just a bit caught up in work. Should be able to catch up with issues and pull requests for this project soon. Thanks for the reminder.

As a note, anyone that wants to step up and help out with maintenance will be greatly welcome.

@angvp
Copy link
Contributor Author

angvp commented Feb 28, 2017

Count me in!

@jpadilla
Copy link
Owner

jpadilla commented Mar 1, 2017

@angvp awesome, thanks! just sent out an invite. Unfortunately Travis CI is still having issues after AWS outage so I can't see what's failing now here after my resolve. Will wait it out.

@angvp
Copy link
Contributor Author

angvp commented Mar 2, 2017

@jpadilla Can you re run travis on this pr? thanks!

@jpadilla
Copy link
Owner

jpadilla commented Mar 2, 2017

@angvp green :)

@angvp angvp merged commit b356a8c into jpadilla:master Mar 2, 2017
@jpadilla jpadilla added this to the 1.10 milestone Mar 22, 2017
@jpadilla
Copy link
Owner

Just released this under v1.10. Thank you @angvp!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants