Skip to content

API 3187 updates to the Python API client #21

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 20, 2016
Merged

API 3187 updates to the Python API client #21

merged 1 commit into from
Jul 20, 2016

Conversation

flaker
Copy link
Contributor

@flaker flaker commented Jul 20, 2016

  • Dropped support for Python 2.4 and 2.5.
  • Simplified code dealing with python versions.
  • General formatting.
  • Setup:
    • Removed the 'oauth2' dependency.
    • No intervention from user is needed to install requests_oauthlib.
    • Updated contact info.

@emilyzzz
Copy link
Contributor

@flaker , can you rebase from current develop? That would remove the 2 extra commits and only leave your own commit.

@flaker flaker force-pushed the API-3187_assorted_fixes branch from aecf8c0 to 0cefb66 Compare July 20, 2016 05:55
@flaker
Copy link
Contributor Author

flaker commented Jul 20, 2016

@emilyzzz done. Thanks.

API-3187 updates to API Python Client
95862bb 
 - Dropped support for Python 2.4 and 2.5.
 - Simplified code dealing with python versions.
 - General:
	- formatting
	- code duplication removal
 - Setup:
	- Removed oauth2
 	- No intervention from user is needed to install requests_oauthlib
	- Updated contact info.
@flaker flaker force-pushed the API-3187_assorted_fixes branch from 0cefb66 to 95862bb Compare July 20, 2016 06:27
@dxtian
Copy link
Contributor

dxtian commented Jul 20, 2016

lgtm

@dxtian dxtian merged commit 742596e into openx:develop Jul 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@flaker @emilyzzz @dxtian