Add flags to describe.py

Add flags to describe.py

[dhermes, 2013-02-28 20:59:52]

https://codereview.appspot.com/7438047/diff/2001/describe.py
File describe.py (right):

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode132
describe.py:132: DISCOVERY_URI =
('https://www.googleapis.com/discovery/v1/apis/'
Why not import this from apiclient.discovery?

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode350
describe.py:350: def document_from_document(uri):
Can we get a better name?

document_api_from_discovery_document?

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode382
describe.py:382: 'https://www.googleapis.com/discovery/v1/apis?preferred=true',
May be worth it to allow this to be a flag as well.

This is part of the discovery API, so it could be someone else's discovery API.

[jcgregorio_google, 2013-03-01 17:22:49]

https://codereview.appspot.com/7438047/diff/2001/describe.py
File describe.py (right):

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode132
describe.py:132: DISCOVERY_URI =
('https://www.googleapis.com/discovery/v1/apis/'
On 2013/02/28 20:59:52, dhermes wrote:
> Why not import this from apiclient.discovery?

Done.

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode350
describe.py:350: def document_from_document(uri):
On 2013/02/28 20:59:52, dhermes wrote:
> Can we get a better name?
> 
> document_api_from_discovery_document?

Done.

https://codereview.appspot.com/7438047/diff/2001/describe.py#newcode382
describe.py:382: 'https://www.googleapis.com/discovery/v1/apis?preferred=true',
On 2013/02/28 20:59:52, dhermes wrote:
> May be worth it to allow this to be a flag as well.
> 
> This is part of the discovery API, so it could be someone else's discovery
API.

Done.

[dhermes, 2013-03-02 00:28:21]

LGTM

[jcgregorio_google, 2013-03-04 01:47:54]

On 2013/03/02 00:28:21, dhermes wrote:
> LGTM

Committed in
https://code.google.com/p/google-api-python-client/source/detail?r=a1a14a7850...