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

migrate-to-requests-4 #5

Merged
merged 3 commits into from
Apr 11, 2012
Merged

Conversation

ntoll
Copy link
Member

@ntoll ntoll commented Apr 11, 2012

This branch introduces one change: it migrates Fluidinfo.py from using httplib2 to the new all singing, all dancing requests library.

Fixes #4

@paparent
Copy link
Member

+1 LGTM! All tests pass.. on Python2.6, distutils doesn't install requirements but it's working in python 2.7.. it's up to you :)

ntoll added a commit that referenced this pull request Apr 11, 2012
branch migrate-to-requests-4: Does exactly what it says on the tin.
author: ntoll
reviewer: paparent
fixes #4
@ntoll ntoll merged commit 8bb60c2 into fluidinfo:master Apr 11, 2012
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.

Drop httplib2 and sit atop requests
2 participants