This repository has been archived by the owner on Nov 8, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 18
python template should use 'requests' #5
Comments
hi, @philfreo - feel free to participate - I will be glad to merge pull request |
We have some proposal, but it unfortunately does not handle all cases. I'll post it here so we can collaborate. |
Almad
added a commit
that referenced
this issue
Mar 11, 2013
@philfreo There is the user-contributed proposal, I just want to do rest of the requests as well (options, head etc.). Do you know if we can just use getattr(requests, method) and it will work in all cases? |
Yup, should work. |
honzajavorek
added a commit
that referenced
this issue
Jul 23, 2014
Aims of these changes are: - simplify the example - introduce the requests library http://docs.python-requests.org/, which is ubiquitous in Python ecosystem these days - introduce Python2/Python3 compatibility (tested on both) - fix minor issues with formatting and indentation - remove unnecessary conversions of body payload according to Content-Type - we already get it as string, we can use it directly as string I suppose This proposal should supersede both https://github.com/apiaryio/language-templates/tree/python-requests and https://github.com/apiaryio/language-templates/tree/python3 branches. It should also fix #5.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
the example python code would be much simpler if it used requests
The text was updated successfully, but these errors were encountered: