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

Updated django-rest-framework version requirement #20

Merged
merged 1 commit into from
Apr 20, 2017

Conversation

rohithasrk
Copy link
Contributor

Fixes #17
3.6.2 seems to be the latest version and all the tests seem to run perfectly.

@coveralls
Copy link

coveralls commented Apr 18, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 2871e3b on rohithasrk:upgrade-rest into 082d236 on netjson:master.

Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

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

Thx @rohithasrk, see my inline comment

requirements.txt Outdated
@@ -1,5 +1,5 @@
django>=1.9,<1.11
djangorestframework>=3.3,<3.6
djangorestframework>=3.3,<=3.6.2
Copy link
Member

@nemesifier nemesifier Apr 19, 2017

Choose a reason for hiding this comment

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

@rohithasrk follow the convention, that is, to put less than the next major version, that is, <3.7, this way we won't have to change this number at each minor release of django-rest-framework 3.6 (eg when 3.6.3 comes out). We assume new minor versions do not bring backward incompatible changes.

Copy link
Contributor Author

@rohithasrk rohithasrk Apr 19, 2017

Choose a reason for hiding this comment

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

Oh. I see @nemesisdesign. Updated 👍

@coveralls
Copy link

coveralls commented Apr 19, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 35760c9 on rohithasrk:upgrade-rest into 082d236 on netjson:master.

@nemesifier nemesifier merged commit ce45caa into openwisp:master Apr 20, 2017
@nemesifier
Copy link
Member

@rohithasrk thanks

@rohithasrk rohithasrk deleted the upgrade-rest branch April 20, 2017 10:15
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