From ef47044dc84762e87d37240df9ea0ad84cf30974 Mon Sep 17 00:00:00 2001
From: Massimiliano Pippi <mpippi@gmail.com>
Date: Tue, 23 Jan 2024 15:20:13 +0100
Subject: [PATCH] linting

---
 pyproject.toml                        |  7 ++++++
 src/haystack_pydoc_tools/loaders.py   |  2 +-
 src/haystack_pydoc_tools/renderers.py | 31 +++++++++++++--------------
 3 files changed, 23 insertions(+), 17 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index d3850e6..214b111 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -157,3 +157,10 @@ exclude_lines = [
   "if __name__ == .__main__.:",
   "if TYPE_CHECKING:",
 ]
+
+[[tool.mypy.overrides]]
+module = [
+  "pydoc_markdown.*",
+  "docspec.*",
+]
+ignore_missing_imports = true
\ No newline at end of file
diff --git a/src/haystack_pydoc_tools/loaders.py b/src/haystack_pydoc_tools/loaders.py
index 0df8344..b81e666 100644
--- a/src/haystack_pydoc_tools/loaders.py
+++ b/src/haystack_pydoc_tools/loaders.py
@@ -2,8 +2,8 @@
 import typing as t
 
 import docspec
-from pydoc_markdown.interfaces import Context
 from pydoc_markdown.contrib.loaders.python import PythonLoader
+from pydoc_markdown.interfaces import Context
 
 
 class CustomPythonLoader(PythonLoader):
diff --git a/src/haystack_pydoc_tools/renderers.py b/src/haystack_pydoc_tools/renderers.py
index 6450f66..59b267e 100644
--- a/src/haystack_pydoc_tools/renderers.py
+++ b/src/haystack_pydoc_tools/renderers.py
@@ -1,17 +1,15 @@
+import base64
+import dataclasses
 import os
 import sys
-import io
-import dataclasses
 import typing as t
-import base64
 import warnings
 from pathlib import Path
 
-import requests
 import docspec
-from pydoc_markdown.interfaces import Context, Renderer
+import requests
 from pydoc_markdown.contrib.renderers.markdown import MarkdownRenderer
-
+from pydoc_markdown.interfaces import Context, Renderer
 
 README_FRONTMATTER = """---
 title: {title}
@@ -30,8 +28,8 @@ def create_headers(version: str):
     # Utility function to create Readme.io headers.
     # We assume the README_API_KEY env var is set since we check outside
     # to show clearer error messages.
-    README_API_KEY = os.getenv("README_API_KEY")
-    token = base64.b64encode(f"{README_API_KEY}:".encode()).decode()
+    api_key = os.getenv("README_API_KEY")
+    token = base64.b64encode(f"{api_key}:".encode()).decode()
     return {"authorization": f"Basic {token}", "x-readme-version": version}
 
 
@@ -78,9 +76,9 @@ def _readme_categories(self, version: str) -> t.Dict[str, str]:
         README_API_KEY env var must be set to correctly get the categories.
         Returns dictionary containing all the categories slugs and their ids.
         """
-        README_API_KEY = os.getenv("README_API_KEY")
-        if not README_API_KEY:
-            warnings.warn("README_API_KEY env var is not set, using a placeholder category ID")
+        api_key = os.getenv("README_API_KEY")
+        if not api_key:
+            warnings.warn("README_API_KEY env var is not set, using a placeholder category ID", stacklevel=2)
             return {"haystack-classes": "ID"}
 
         headers = create_headers(version)
@@ -103,9 +101,9 @@ def _doc_id(self, doc_slug: str, version: str) -> str:
             # we just return an empty string.
             return ""
 
-        README_API_KEY = os.getenv("README_API_KEY")
-        if not README_API_KEY:
-            warnings.warn("README_API_KEY env var is not set, using a placeholder doc ID")
+        api_key = os.getenv("README_API_KEY")
+        if not api_key:
+            warnings.warn("README_API_KEY env var is not set, using a placeholder doc ID", stacklevel=2)
             return "fake-doc-id"
 
         headers = create_headers(version)
@@ -120,7 +118,7 @@ def render(self, modules: t.List[docspec.Module]) -> None:
             sys.stdout.write(self._frontmatter())
             self.markdown.render_single_page(sys.stdout, modules)
         else:
-            with io.open(self.markdown.filename, "w", encoding=self.markdown.encoding) as fp:
+            with open(self.markdown.filename, "w", encoding=self.markdown.encoding) as fp:
                 fp.write(self._frontmatter())
                 self.markdown.render_single_page(t.cast(t.TextIO, fp), modules)
 
@@ -138,7 +136,8 @@ def _frontmatter(self) -> str:
 @dataclasses.dataclass
 class ReadmePreviewRenderer(ReadmeRenderer):
     """
-    This custom Renderer behaves just like the ReadmeRenderer but renders docs with the hardcoded version 2.0 to generate correct category ids.
+    This custom Renderer behaves just like the ReadmeRenderer but renders docs with the hardcoded
+    version 2.0 to generate correct category ids.
     """
 
     def _doc_version(self) -> str: