LGTM apart from the potential Buzz breakage. http://codereview.appspot.com/4515144/diff/3001/apiclient/discovery.py File apiclient/discovery.py (right): http://codereview.appspot.com/4515144/diff/3001/apiclient/discovery.py#newcod... apiclient/discovery.py:350: expanded_url = uritemplate.expand(pathUrl, params) This change removes the workaround for the Buzz discovery document bug. Is that actually fixed?
That should have been fixed a long time ago, is there a path that looks like it won't work? On Thu, May 26, 2011 at 8:08 PM, <ade@google.com> wrote: > LGTM apart from the potential Buzz breakage. > > > http://codereview.appspot.com/4515144/diff/3001/apiclient/discovery.py > File apiclient/discovery.py (right): > > http://codereview.appspot.com/4515144/diff/3001/apiclient/discovery.py#newcod... > apiclient/discovery.py:350: expanded_url = uritemplate.expand(pathUrl, > params) > This change removes the workaround for the Buzz discovery document bug. > Is that actually fixed? > > http://codereview.appspot.com/4515144/ > -- Joe Gregorio http://bitworking.org