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

Using latest pip installed version of requests breaks library completely #1

Open
eads opened this issue Jan 4, 2013 · 5 comments
Open
Labels

Comments

@eads
Copy link
Collaborator

eads commented Jan 4, 2013

using requests==1.0.4, the get(...) method passes an unexpected param called config. Downgrading to requests 0.13.2 (the last version I used which I knew worked with the library) fixes the problem. Seems like requests has changed it's allowed parameters since the summer.

@andymboyle
Copy link
Contributor

I believe I have fixed this.

@eads
Copy link
Collaborator Author

eads commented Jan 9, 2013

I fixed it, but I doubt the fixes are backwards compatible to old Requests versions. Will we want that, or can we call it fixed?

@rnagle
Copy link
Contributor

rnagle commented Jan 9, 2013

I think we can call it fixed as long as the tests pass.

@eads
Copy link
Collaborator Author

eads commented Jan 9, 2013

Do you know how to run the tests? Do we just need to set environment
variables for the test run?

On Wed, Jan 9, 2013 at 2:52 PM, Ryan Nagle notifications@github.com wrote:

I think we can call it fixed as long as the tests pass.


Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-12066002.

David Eads - 773.354.2285
News applications developer, Chicago Tribune (
http://www.chicagotribune.com/news/data/)
Founder, FreeGeek Chicago (http://freegeekchicago.org)

“The awful thing about life is this: everyone has their reasons.” -- Jean
Renoir

@rnagle
Copy link
Contributor

rnagle commented Jan 9, 2013

Make sure you have P2P_API_URL and P2P_API_KEY env vars set. Create a new virtualenv for the p2p module and install the module in it. Then run the tests: cd path/to/p2p-python; ./p2p/tests.py

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

No branches or pull requests

3 participants