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

Adapt to botocore pr 645 #271

Merged
merged 3 commits into from
Sep 24, 2015
Merged

Conversation

rayluo
Copy link
Contributor

@rayluo rayluo commented Sep 23, 2015

This PR is a quick patch to work with boto/botocore#645

All the commits here are duplicated in pending PR #239

@rayluo
Copy link
Contributor Author

rayluo commented Sep 23, 2015

Well, forget the 3rd commit. assert_in is not supported in python 2.6. I will take it away in next PR.

@@ -19,6 +19,12 @@
from boto3.docs.service import ServiceDocumenter


# This is a temporary hack
def _snake2camel(s):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the reason for adding this as opposed to what I did in my pending PR? I know you labeled it as temporary, but it seems super fragile.

@kyleknap
Copy link
Contributor

🚢

rayluo added a commit that referenced this pull request Sep 24, 2015
@rayluo rayluo merged commit 24df8ca into boto:develop Sep 24, 2015
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.

2 participants