chore: fix F841 errors reported by flake8

JohnVillalovos · JohnVillalovos · commit 70f3dae268eb · 2021-04-18T07:13:56.000-07:00
Local variable name is assigned to but never used https://www.flake8rules.com/rules/F841.html
diff --git a/gitlab/cli.py b/gitlab/cli.py
@@ -162,7 +162,6 @@ def docs() -> argparse.ArgumentParser:
     if "sphinx" not in sys.modules:
         sys.exit("Docs parser is only intended for build_sphinx")
 
-    parser = _get_base_parser(add_help=False)
     # NOTE: We must delay import of gitlab.v4.cli until now or
     # otherwise it will cause circular import errors
     import gitlab.v4.cli
diff --git a/gitlab/tests/objects/test_appearance.py b/gitlab/tests/objects/test_appearance.py
@@ -63,4 +63,4 @@ def test_get_update_appearance(gl, resp_application_appearance):
 
 
 def test_update_appearance(gl, resp_application_appearance):
-    resp = gl.appearance.update(title=new_title, description=new_description)
+    gl.appearance.update(title=new_title, description=new_description)
diff --git a/gitlab/tests/test_base.py b/gitlab/tests/test_base.py
@@ -92,7 +92,7 @@ def test_picklability(self, fake_manager):
         assert isinstance(unpickled, FakeObject)
         assert hasattr(unpickled, "_module")
         assert unpickled._module == original_obj_module
-        pickled2 = pickle.dumps(unpickled)
+        pickle.dumps(unpickled)
 
     def test_attrs(self, fake_manager):
         obj = FakeObject(fake_manager, {"foo": "bar"})
diff --git a/gitlab/v4/objects/todos.py b/gitlab/v4/objects/todos.py
@@ -48,4 +48,4 @@ def mark_all_as_done(self, **kwargs):
         Returns:
             int: The number of todos maked done
         """
-        result = self.gitlab.http_post("/todos/mark_as_done", **kwargs)
+        self.gitlab.http_post("/todos/mark_as_done", **kwargs)

Original file line number	Diff line number	Diff line change
`@@ -63,4 +63,4 @@ def test_get_update_appearance(gl, resp_application_appearance):`
`63`	`63`
`64`	`64`
`65`	`65`	`def test_update_appearance(gl, resp_application_appearance):`
`66`		`- resp = gl.appearance.update(title=new_title, description=new_description)`
	`66`	`+ gl.appearance.update(title=new_title, description=new_description)`