On 2012/12/07 23:15:58, Ali Afshar wrote: > On 2012/12/07 20:45:54, jcgregorio_google wrote: > > Consider ...
11 years, 9 months ago
(2012-12-10 14:02:14 UTC)
#3
On 2012/12/07 23:15:58, Ali Afshar wrote:
> On 2012/12/07 20:45:54, jcgregorio_google wrote:
>
> Consider refactoring so that we have two separate functions, one from a string
> and one from a dict?
Seems a little heavy handed to have a new function for that as it is still
building from a discovery document, we just accept the serialized and the
deserialized form. A new function would be appropriate if it was building a
service object from something completely different, for example, we may be
adding a build_from_cache() function.
On 2012/12/10 14:02:14, jcgregorio_google wrote: > On 2012/12/07 23:15:58, Ali Afshar wrote: > > On ...
11 years, 9 months ago
(2012-12-10 14:45:49 UTC)
#4
On 2012/12/10 14:02:14, jcgregorio_google wrote:
> On 2012/12/07 23:15:58, Ali Afshar wrote:
> > On 2012/12/07 20:45:54, jcgregorio_google wrote:
> >
> > Consider refactoring so that we have two separate functions, one from a
string
> > and one from a dict?
>
> Seems a little heavy handed to have a new function for that as it is still
> building from a discovery document, we just accept the serialized and the
> deserialized form. A new function would be appropriate if it was building a
> service object from something completely different, for example, we may be
> adding a build_from_cache() function.
As you wish. I thought it would be convenient to have a single line function,
while not relying on the type check.
Committed in http://code.google.com/p/google-api-python-client/source/detail?r=ed1095576a26de8fecdadbb8ae322a6b063cce4d On 2012/12/10 14:45:49, Ali Afshar wrote: > On 2012/12/10 14:02:14, jcgregorio_google wrote: ...
11 years, 9 months ago
(2012-12-10 15:25:27 UTC)
#5
Committed in
http://code.google.com/p/google-api-python-client/source/detail?r=ed1095576a2...
On 2012/12/10 14:45:49, Ali Afshar wrote:
> On 2012/12/10 14:02:14, jcgregorio_google wrote:
> > On 2012/12/07 23:15:58, Ali Afshar wrote:
> > > On 2012/12/07 20:45:54, jcgregorio_google wrote:
> > >
> > > Consider refactoring so that we have two separate functions, one from a
> string
> > > and one from a dict?
> >
> > Seems a little heavy handed to have a new function for that as it is still
> > building from a discovery document, we just accept the serialized and the
> > deserialized form. A new function would be appropriate if it was building a
> > service object from something completely different, for example, we may be
> > adding a build_from_cache() function.
>
> As you wish. I thought it would be convenient to have a single line function,
> while not relying on the type check.
Issue 6906052: Allow deserialized discovery docs to be passed to build_from_document().
(Closed)
Created 11 years, 9 months ago by jcgregorio_google
Modified 11 years, 9 months ago
Reviewers: Ali Afshar
Base URL:
Comments: 0