Skip to content
This repository was archived by the owner on Oct 24, 2022. It is now read-only.

Update requests requirement with API changes.#35

Merged
copitux merged 1 commit into
copitux:masterfrom
DataDog:upgrade-requests
Oct 23, 2014
Merged

Update requests requirement with API changes.#35
copitux merged 1 commit into
copitux:masterfrom
DataDog:upgrade-requests

Conversation

@conorbranagan

Copy link
Copy Markdown

We're currently having conflicts with the requests requirement because it's pinned to an old version. Tests are passing after making a couple small changes where the requests API has changed.

Related to #33.

@accorvin

Copy link
Copy Markdown

I support merging this pull request. I independently made basically the same change while trying to get this project to work with another pip package. I've noticed no behavioral changes using the new version of requests, and all tests still pass.

@rhlobo

rhlobo commented Mar 7, 2014

Copy link
Copy Markdown

+1 Please merge this pull-request.

@intellisense

Copy link
Copy Markdown

+1 Please merge this pull-request

@ray-dino

Copy link
Copy Markdown

Please merge this pull request. Github has disabled support for SSLv3. Using the latest version of python requests is critical now.

@ray-dino ray-dino mentioned this pull request Oct 22, 2014
@copitux

copitux commented Oct 23, 2014

Copy link
Copy Markdown
Owner

Merged. I'd like to review this lib at this weekend

copitux added a commit that referenced this pull request Oct 23, 2014
Update requests requirement with API changes.
@copitux copitux merged commit 1fe22b8 into copitux:master Oct 23, 2014
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.

6 participants