code/twython

Fork 0

Update all endpoints in the api_table, examples, READMEs #188

Merged

michaelhelmick merged 4 commits from pep8-functions into master

2013-05-04 16:48:03 -07:00

michaelhelmick commented

2013-05-04 16:29:00 -07:00

(Migrated from github.com)

Also updated functions in twython.py to use pep8 functions instead of
camelCase functions

Also updated functions in twython.py to use pep8 functions instead of camelCase functions

ryanmcgrath commented

2013-05-04 16:31:24 -07:00

(Migrated from github.com)

Haaa, I was about to commit this same stuff. You beat me to it! >_<;

Perfect work, I'd say. Uber excited to see this. Merge and release?

Haaa, I was about to commit this same stuff. You beat me to it! >_<; Perfect work, I'd say. Uber excited to see this. Merge and release?

michaelhelmick commented

2013-05-04 16:32:44 -07:00

(Migrated from github.com)

@ryanmcgrath Old methods still work and fallback gracefully accompanied with a TwythonDeprecationWarning

All media methods and other methods that weren't pep8'd already in twython.py I updated and their old camelCase version works still but gives a TwythonDeprecationWarning

There are a couple methods that didn't get hit by that replace fix... i.e. getFollowersIDs because IDs was capitalized instead of getFollowerIds

There was like 3 or 4 of those functions, ideas on what we should do about that?

@ryanmcgrath Old methods still work and fallback gracefully accompanied with a `TwythonDeprecationWarning` All media methods and other methods that weren't pep8'd already in `twython.py` I updated and their old camelCase version works still but gives a `TwythonDeprecationWarning` There are a couple methods that _didn't_ get hit by that replace fix... i.e. `getFollowersIDs` because IDs was capitalized instead of `getFollowerIds` There was like 3 or 4 of those functions, ideas on what we should do about that?

ryanmcgrath commented

2013-05-04 16:36:00 -07:00

(Migrated from github.com)

Hmm, do we have a full list of those? If it's just 3 or 4, specifically check for those? The only other somewhat clean alternative I could think of would be to remove them from the table and set them up as full definitions in twython.py and then they get their cases handled specially...

Hmm, do we have a full list of those? If it's just 3 or 4, specifically check for those? The only other somewhat clean alternative I could think of would be to remove them from the table and set them up as full definitions in `twython.py` and then they get their cases handled specially...

michaelhelmick commented

2013-05-04 16:36:57 -07:00

(Migrated from github.com)

Yeah, I'll get the list, they we're all in a row. Let me check the diff and commit

Mike Helmick
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Saturday, May 4, 2013 at 7:36 PM, Ryan McGrath wrote:

Hmm, do we have a full list of those? If it's just 3 or 4, specifically check for those? The only other somewhat clean alternative I could think of would be to remove them from the table and set them up as full definitions in twython.py and then they get their cases handled specially...

—
Reply to this email directly or view it on GitHub (https://github.com/ryanmcgrath/twython/pull/188#issuecomment-17443217).

Yeah, I'll get the list, they we're all in a row. Let me check the diff and commit ## Mike Helmick Sent with Sparrow (http://www.sparrowmailapp.com/?sig) On Saturday, May 4, 2013 at 7:36 PM, Ryan McGrath wrote: > Hmm, do we have a full list of those? If it's just 3 or 4, specifically check for those? The only other somewhat clean alternative I could think of would be to remove them from the table and set them up as full definitions in twython.py and then they get their cases handled specially... > > — > Reply to this email directly or view it on GitHub (https://github.com/ryanmcgrath/twython/pull/188#issuecomment-17443217).

No reviewers

No labels

Bug

Enhancement

Feature Suggestion

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#188

No description provided.

Rows
Columns