cc

gitea/apiobject.py

@@ -28,7 +28,7 @@ class Organization(ApiObject):
         return hash(self.gitea) ^ hash(self.name)
 
     @classmethod
-    def request(cls, gitea: 'Gitea', name: str):
+    def request(cls, gitea: 'Gitea', name: str) -> 'Organization':
         return cls._request(gitea, {"name": name})
 
     @classmethod
@@ -363,7 +363,7 @@ class Repository(ApiObject):
     def get_full_name(self) -> str:
         return self.owner.username + "/" + self.name
 
-    def create_issue(self, title, assignees=[], description="") -> ApiObject:
+    def create_issue(self, title, assignees=frozenset(), description="") -> ApiObject:
         data = {
             "assignees": assignees,
             "body": description,
@@ -542,7 +542,7 @@ class Commit(ReadonlyApiObject):
         return hash(self.sha)
 
     @classmethod
-    def parse_response(cls, gitea, result):
+    def parse_response(cls, gitea, result) -> 'Commit':
         commit_cache = result["commit"]
         api_object = cls(gitea)
         cls._initialize(gitea, api_object, result)

gitea/baseapiobject.py

@@ -47,7 +47,6 @@ class ReadonlyApiObject:
 
     @classmethod
     def _initialize(cls, gitea, api_object, result):
-        x = 1
         for name, value in result.items():
             if name in cls._fields_to_parsers and value is not None:
                 parse_func = cls._fields_to_parsers[name]
@@ -63,7 +62,7 @@ class ReadonlyApiObject:
         if not hasattr(api_object, name):
             setattr(api_object, "_" + name, value)
             prop = property(
-                (lambda name: lambda self: self._get_var(name))(name))
+                (lambda n: lambda self: self._get_var(n))(name))
             setattr(cls, name, prop)
         else:
             raise AttributeError(f"Attribute {name} already exists on api object.")
@@ -107,8 +106,8 @@ class ApiObject(ReadonlyApiObject):
         if not hasattr(api_object, "_" + name):
             setattr(api_object, "_" + name, value)
         prop = property(
-            (lambda name: lambda self: self._get_var(name))(name),
-            (lambda name: lambda self, v: self.__set_var(name, v))(name))
+            (lambda n: lambda self: self._get_var(n))(name),
+            (lambda n: lambda self, v: self.__set_var(n, v))(name))
         setattr(cls, name, prop)
 
     def __set_var(self, name, i):

requirements.txt

@@ -1,3 +1,4 @@
 requests
 httpcache
 pytest
+frozendict