Skip to content
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

requests > 2.4.2 has an optional .post json arg #28

Merged
merged 2 commits into from
Apr 1, 2015

Conversation

zoidyzoidzoid
Copy link
Contributor

Requests changelog

There haven't been any huge API changes since version one of requests. We could probably support >=1.0.0, <3.0.0.

@stefanor
Copy link

👍 Seems reasonable. But consumer requirements will probably need updating, too.

@zoidyzoidzoid
Copy link
Contributor Author

Minor version bump with a warning or a major version bump in case people are fiddling with requests
internals?

@stefanor
Copy link

Minor is fine.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.14% when pulling 0edb70a on requests-2-support into e393064 on master.

@RayeN
Copy link
Member

RayeN commented Apr 1, 2015

👍

zoidyzoidzoid added a commit that referenced this pull request Apr 1, 2015
requests > 2.4.2 has an optional .post json arg
@zoidyzoidzoid zoidyzoidzoid merged commit fc042cf into master Apr 1, 2015
@zoidyzoidzoid zoidyzoidzoid deleted the requests-2-support branch April 1, 2015 07:42
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.

None yet

4 participants