Merge pull request #2210 from python-gitlab/jlvillal/mypy_step_by_step

chore: enable mypy check `no_implicit_optional`

gitlab/v4/objects/appearance.py

@@ -38,7 +38,7 @@ class ApplicationAppearanceManager(GetWithoutIdMixin, UpdateMixin, RESTManager):
     def update(
         self,
         id: Optional[Union[str, int]] = None,
-        new_data: Dict[str, Any] = None,
+        new_data: Optional[Dict[str, Any]] = None,
         **kwargs: Any
     ) -> Dict[str, Any]:
         """Update an object on the server.

gitlab/v4/objects/repositories.py

@@ -196,7 +196,7 @@ def repository_contributors(
     @exc.on_http_error(exc.GitlabListError)
     def repository_archive(
         self,
-        sha: str = None,
+        sha: Optional[str] = None,
         streamed: bool = False,
         action: Optional[Callable[..., Any]] = None,
         chunk_size: int = 1024,

gitlab/v4/objects/settings.py

@@ -93,7 +93,7 @@ class ApplicationSettingsManager(GetWithoutIdMixin, UpdateMixin, RESTManager):
     def update(
         self,
         id: Optional[Union[str, int]] = None,
-        new_data: Dict[str, Any] = None,
+        new_data: Optional[Dict[str, Any]] = None,
         **kwargs: Any
     ) -> Dict[str, Any]:
         """Update an object on the server.

pyproject.toml

@@ -13,6 +13,7 @@ disallow_incomplete_defs = true
 disallow_subclassing_any = true
 disallow_untyped_decorators = true
 disallow_untyped_defs = true
+no_implicit_optional = true
 no_implicit_reexport = true
 strict_equality = true
 warn_redundant_casts = true
@@ -23,7 +24,6 @@ warn_unused_ignores = true
 # The following need to have changes made to be able to enable them:
 # disallow_any_generics = true
 # disallow_untyped_calls = true
-# no_implicit_optional = true
 
 [[tool.mypy.overrides]] # Overrides for currently untyped modules
 module = [