chore: remove old-style classes

nejch · JohnVillalovos · commit ae2a015db101 · 2022-01-23T07:14:49.000-08:00
diff --git a/gitlab/base.py b/gitlab/base.py
@@ -41,7 +41,7 @@
 )
 
 
-class RESTObject(object):
+class RESTObject:
     """Represents an object built from server data.
 
     It holds the attributes know from the server, and the updated attributes in
@@ -234,7 +234,7 @@ def attributes(self) -> Dict[str, Any]:
         return d
 
 
-class RESTObjectList(object):
+class RESTObjectList:
     """Generator object representing a list of RESTObject's.
 
     This generator uses the Gitlab pagination system to fetch new data when
@@ -321,7 +321,7 @@ class RequiredOptional(NamedTuple):
     optional: Tuple[str, ...] = tuple()
 
 
-class RESTManager(object):
+class RESTManager:
     """Base class for CRUD operations on objects.
 
     Derived class must define ``_path`` and ``_obj_cls``.
diff --git a/gitlab/client.py b/gitlab/client.py
@@ -36,7 +36,7 @@
 )
 
 
-class Gitlab(object):
+class Gitlab:
     """Represents a GitLab server connection.
 
     Args:
@@ -957,7 +957,7 @@ def search(
         return self.http_list("/search", query_data=data, **kwargs)
 
 
-class GitlabList(object):
+class GitlabList:
     """Generator representing a list of remote objects.
 
     The object handles the links returned by a query to the API, and will call
diff --git a/gitlab/config.py b/gitlab/config.py
@@ -101,7 +101,7 @@ class GitlabConfigHelperError(ConfigError):
     pass
 
 
-class GitlabConfigParser(object):
+class GitlabConfigParser:
     def __init__(
         self, gitlab_id: Optional[str] = None, config_files: Optional[List[str]] = None
     ) -> None:
diff --git a/gitlab/types.py b/gitlab/types.py
@@ -18,7 +18,7 @@
 from typing import Any, Optional, TYPE_CHECKING
 
 
-class GitlabAttribute(object):
+class GitlabAttribute:
     def __init__(self, value: Any = None) -> None:
         self._value = value
 
diff --git a/gitlab/utils.py b/gitlab/utils.py
@@ -21,7 +21,7 @@
 import requests
 
 
-class _StdoutStream(object):
+class _StdoutStream:
     def __call__(self, chunk: Any) -> None:
         print(chunk)
 
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
@@ -27,7 +27,7 @@
 from gitlab import cli
 
 
-class GitlabCLI(object):
+class GitlabCLI:
     def __init__(
         self, gl: gitlab.Gitlab, what: str, action: str, args: Dict[str, str]
     ) -> None:
@@ -359,7 +359,7 @@ def get_dict(
     return obj.attributes
 
 
-class JSONPrinter(object):
+class JSONPrinter:
     def display(self, d: Union[str, Dict[str, Any]], **kwargs: Any) -> None:
         import json  # noqa
 
@@ -376,7 +376,7 @@ def display_list(
         print(json.dumps([get_dict(obj, fields) for obj in data]))
 
 
-class YAMLPrinter(object):
+class YAMLPrinter:
     def display(self, d: Union[str, Dict[str, Any]], **kwargs: Any) -> None:
         try:
             import yaml  # noqa
@@ -411,7 +411,7 @@ def display_list(
             )
 
 
-class LegacyPrinter(object):
+class LegacyPrinter:
     def display(self, d: Union[str, Dict[str, Any]], **kwargs: Any) -> None:
         verbose = kwargs.get("verbose", False)
         padding = kwargs.get("padding", 0)
diff --git a/tests/meta/test_mro.py b/tests/meta/test_mro.py
@@ -20,7 +20,7 @@ class ProjectPipeline(RefreshMixin, ObjectDeleteMixin, RESTObject):
 
   Here is how our classes look when type-checking:
 
-      class RESTObject(object):
+      class RESTObject:
           def __init__(self, manager: "RESTManager", attrs: Dict[str, Any]) -> None:
               ...
 
@@ -52,7 +52,7 @@ class Wrongv4Object(RESTObject, Mixin):
 def test_show_issue() -> None:
     """Test case to demonstrate the TypeError that occurs"""
 
-    class RESTObject(object):
+    class RESTObject:
         def __init__(self, manager: str, attrs: int) -> None:
             ...
 
diff --git a/tests/unit/test_base.py b/tests/unit/test_base.py
@@ -23,7 +23,7 @@
 from gitlab import base
 
 
-class FakeGitlab(object):
+class FakeGitlab:
     pass
 
 
@@ -61,7 +61,7 @@ class MGR(base.RESTManager):
             _obj_cls = object
             _from_parent_attrs = {"test_id": "id"}
 
-        class Parent(object):
+        class Parent:
             id = 42
 
         mgr = MGR(FakeGitlab(), parent=Parent())
