Skip to content
This repository was archived by the owner on Jan 18, 2025. It is now read-only.

Conversation

@theacodes
Copy link
Contributor

Fixes #326, follow-up PRs will be submitted to move flask_util, appengine, and django_orm.

@nathanielmanistaatgoogle
Copy link
Contributor

Looks good to me; waiting on test results.

@theacodes
Copy link
Contributor Author

@dhermes' tricky docs thing got me again. It should pass tests now.

@dhermes
Copy link
Contributor

dhermes commented Dec 1, 2015

Do you see a less tricky way to make sure that contributors that add new code also document their new code?

@theacodes
Copy link
Contributor Author

No, I think it's great. I've just been dumb enough to let me bite me twice now. :)

@theacodes theacodes force-pushed the contrib branch 2 times, most recently from 97e7fe9 to ae89556 Compare December 1, 2015 18:32
@dhermes
Copy link
Contributor

dhermes commented Dec 1, 2015

👍 😀

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@theacodes
Copy link
Contributor Author

@nathanielmanistaatgoogle ping - travis is green. :)

nathanielmanistaatgoogle added a commit that referenced this pull request Dec 4, 2015
Added empty oauth2client.contrib for contributed modules.
@nathanielmanistaatgoogle nathanielmanistaatgoogle merged commit a0f79a5 into googleapis:master Dec 4, 2015
@dhermes dhermes mentioned this pull request Jan 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants