You and Andy should talk about merging you samples and his samples together. http://codereview.appspot.com/3853041/diff/1/apiclient/errors.py File apiclient/errors.py (right): http://codereview.appspot.com/3853041/diff/1/apiclient/errors.py#newcode27 apiclient/errors.py:27: return str(self.detail) I updated from your branch where you fixed up HttpError, so this should be unnecessary. http://codereview.appspot.com/3853041/diff/1/samples/searchforshopping/rankin... File samples/searchforshopping/ranking.py (right): http://codereview.appspot.com/3853041/diff/1/samples/searchforshopping/rankin... samples/searchforshopping/ranking.py:6: One blank line only. Also below. Needs to be fixed across all the samples. http://codereview.appspot.com/3853041/diff/1/samples/searchforshopping/rankin... samples/searchforshopping/ranking.py:9: One blank line only. http://codereview.appspot.com/3853041/diff/1/samples/searchforshopping/rankin... samples/searchforshopping/ranking.py:12: One blank line only.
LGTM. Committed on 149:45f58fa78fd0 On 2011/01/05 12:52:48, Ali Afshar wrote: