Improve comformance to pep8 (+dos2unix)

PyGithub is now a public projects with several public contibutors,
I have to abandon my very personal coding conventions.
This commit is contained in:
Vincent Jacques
2012-09-17 20:37:22 +02:00
parent 7f61adcaf5
commit 531c3f2d32
107 changed files with 5245 additions and 5094 deletions
+35 -33
View File
@@ -24,21 +24,23 @@ import Legacy
import GithubObject
import HookDescription
DEFAULT_BASE_URL = "https://api.github.com"
DEFAULT_TIMEOUT = 10
class Github( object ):
def __init__( self, login_or_token = None, password = None, base_url = DEFAULT_BASE_URL, timeout = DEFAULT_TIMEOUT):
self.__requester = Requester( login_or_token, password, base_url, timeout )
class Github(object):
def __init__(self, login_or_token=None, password=None, base_url=DEFAULT_BASE_URL, timeout=DEFAULT_TIMEOUT):
self.__requester = Requester(login_or_token, password, base_url, timeout)
@property
def rate_limiting( self ):
def rate_limiting(self):
return self.__requester.rate_limiting
def get_user( self, login = GithubObject.NotSet ):
assert login is GithubObject.NotSet or isinstance( login, ( str, unicode ) ), login
def get_user(self, login=GithubObject.NotSet):
assert login is GithubObject.NotSet or isinstance(login, (str, unicode)), login
if login is GithubObject.NotSet:
return AuthenticatedUser.AuthenticatedUser( self.__requester, { "url": "/user" }, completed = False )
return AuthenticatedUser.AuthenticatedUser(self.__requester, {"url": "/user"}, completed=False)
else:
headers, data = self.__requester.requestAndCheck(
"GET",
@@ -46,29 +48,29 @@ class Github( object ):
None,
None
)
return NamedUser.NamedUser( self.__requester, data, completed = True )
return NamedUser.NamedUser(self.__requester, data, completed=True)
def get_organization( self, login ):
assert isinstance( login, ( str, unicode ) ), login
def get_organization(self, login):
assert isinstance(login, (str, unicode)), login
headers, data = self.__requester.requestAndCheck(
"GET",
"/orgs/" + login,
None,
None
)
return Organization.Organization( self.__requester, data, completed = True )
return Organization.Organization(self.__requester, data, completed=True)
def get_gist( self, id ):
assert isinstance( id, ( str, unicode ) ), id
def get_gist(self, id):
assert isinstance(id, (str, unicode)), id
headers, data = self.__requester.requestAndCheck(
"GET",
"/gists/" + id,
None,
None
)
return Gist.Gist( self.__requester, data, completed = True )
return Gist.Gist(self.__requester, data, completed=True)
def get_gists( self ):
def get_gists(self):
return PaginatedList.PaginatedList(
Gist.Gist,
self.__requester,
@@ -76,12 +78,12 @@ class Github( object ):
None
)
def legacy_search_repos( self, keyword, language = GithubObject.NotSet ):
assert isinstance( keyword, ( str, unicode ) ), keyword
assert language is GithubObject.NotSet or isinstance( language, ( str, unicode ) ), language
args = {} if language is GithubObject.NotSet else { "language": language }
def legacy_search_repos(self, keyword, language=GithubObject.NotSet):
assert isinstance(keyword, (str, unicode)), keyword
assert language is GithubObject.NotSet or isinstance(language, (str, unicode)), language
args = {} if language is GithubObject.NotSet else {"language": language}
return Legacy.PaginatedList(
"/legacy/repos/search/" + urllib.quote( keyword ),
"/legacy/repos/search/" + urllib.quote(keyword),
args,
self.__requester,
"repositories",
@@ -89,10 +91,10 @@ class Github( object ):
Repository.Repository,
)
def legacy_search_users( self, keyword ):
assert isinstance( keyword, ( str, unicode ) ), keyword
def legacy_search_users(self, keyword):
assert isinstance(keyword, (str, unicode)), keyword
return Legacy.PaginatedList(
"/legacy/user/search/" + urllib.quote( keyword ),
"/legacy/user/search/" + urllib.quote(keyword),
{},
self.__requester,
"users",
@@ -100,25 +102,25 @@ class Github( object ):
NamedUser.NamedUser,
)
def legacy_search_user_by_email( self, email ):
assert isinstance( email, ( str, unicode ) ), email
def legacy_search_user_by_email(self, email):
assert isinstance(email, (str, unicode)), email
headers, data = self.__requester.requestAndCheck(
"GET",
"/legacy/user/email/" + email,
None,
None
)
return NamedUser.NamedUser( self.__requester, Legacy.convertUser( data[ "user" ] ), completed = False )
return NamedUser.NamedUser(self.__requester, Legacy.convertUser(data["user"]), completed=False)
def render_markdown( self, text, context = GithubObject.NotSet ):
assert isinstance( text, ( str, unicode ) ), text
assert context is GithubObject.NotSet or isinstance( context, Repository.Repository ), context
def render_markdown(self, text, context=GithubObject.NotSet):
assert isinstance(text, (str, unicode)), text
assert context is GithubObject.NotSet or isinstance(context, Repository.Repository), context
post_parameters = {
"text": text
}
if context is not GithubObject.NotSet:
post_parameters[ "mode" ] = "gfm"
post_parameters[ "context" ] = context._identity
post_parameters["mode"] = "gfm"
post_parameters["context"] = context._identity
status, headers, data = self.__requester.requestRaw(
"POST",
"/markdown",
@@ -127,11 +129,11 @@ class Github( object ):
)
return data
def get_hooks( self ):
def get_hooks(self):
headers, data = self.__requester.requestAndCheck(
"GET",
"/hooks",
None,
None
)
return [ HookDescription.HookDescription( self.__requester, attributes, completed = True ) for attributes in data ]
return [HookDescription.HookDescription(self.__requester, attributes, completed=True) for attributes in data]