LGTM, just remove unused import (lint?) https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py File apiclient/discovery.py (left): https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode35 apiclient/discovery.py:35: import urllib unused import https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode36 apiclient/discovery.py:36: import urlparse unused import https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode41 apiclient/discovery.py:41: from urlparse import parse_qsl unused import https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode43 apiclient/discovery.py:43: from cgi import parse_qsl unused import
Actually all those imports are used. On 2013/03/06 14:45:46, proppy wrote: > LGTM, just remove unused import (lint?) > > https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py > File apiclient/discovery.py (left): > > https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode35 > apiclient/discovery.py:35: import urllib > unused import > > https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode36 > apiclient/discovery.py:36: import urlparse > unused import > > https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode41 > apiclient/discovery.py:41: from urlparse import parse_qsl > unused import > > https://codereview.appspot.com/7506043/diff/1/apiclient/discovery.py#oldcode43 > apiclient/discovery.py:43: from cgi import parse_qsl > unused import