docsig 0.49.2__tar.gz → 0.50.0__tar.gz

{docsig-0.49.2 → docsig-0.50.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: docsig
-Version: 0.49.2
+Version: 0.50.0
 Summary: Check signature params for proper documentation
 Home-page: https://pypi.org/project/docsig/
 License: MIT
@@ -223,7 +223,7 @@ It can be added to your .pre-commit-config.yaml as follows:
 
     repos:
       - repo: https://github.com/jshwi/docsig
-        rev: v0.49.2
+        rev: v0.50.0
         hooks:
           - id: docsig
             args:

{docsig-0.49.2 → docsig-0.50.0}/README.rst

@@ -195,7 +195,7 @@ It can be added to your .pre-commit-config.yaml as follows:
 
     repos:
       - repo: https://github.com/jshwi/docsig
-        rev: v0.49.2
+        rev: v0.50.0
         hooks:
           - id: docsig
             args:

{docsig-0.49.2 → docsig-0.50.0}/docsig/_core.py

@@ -191,6 +191,7 @@ def docsig(  # pylint: disable=too-many-locals,too-many-arguments
         ignore_args=ignore_args,
         ignore_kwargs=ignore_kwargs,
         check_class_constructor=check_class_constructor,
+        no_ansi=no_ansi,
         verbose=verbose,
     )
     display = _Display(no_ansi)
@@ -225,4 +226,4 @@ def docsig(  # pylint: disable=too-many-locals,too-many-arguments
     else:
         display.report()
 
-    return int(bool(display))
+    return max(int(bool(display)), modules.retcode)

docsig-0.50.0/docsig/_hooks.py

@@ -0,0 +1,20 @@
+"""
+docsig._hooks
+=============
+"""
+
+import sys as _sys
+from os import environ as _e
+
+from ._utils import pretty_print_error
+
+
+def excepthook(no_ansi: bool = False) -> None:
+    """Print user friendly commandline error if debug not enabled.
+
+    :param no_ansi: Disable ANSI output.
+    """
+    if _e.get("DOCSIG_DEBUG", None) != "1":
+        _sys.excepthook = lambda x, y, _: pretty_print_error(
+            x, str(y), no_ansi
+        )

{docsig-0.49.2 → docsig-0.50.0}/docsig/_main.py

@@ -9,7 +9,7 @@ from __future__ import annotations as _
 
 from ._config import Parser as _Parser
 from ._core import docsig as _docsig
-from ._hooks import pretty_print_error as _pretty_print_error
+from ._hooks import excepthook as _excepthook
 
 
 def main() -> str | int:
@@ -20,7 +20,7 @@ def main() -> str | int:
     :return: Exit status for whether test failed or not.
     """
     p = _Parser()
-    _pretty_print_error(p.args.no_ansi)
+    _excepthook(p.args.no_ansi)
     return _docsig(
         *p.args.path,
         string=p.args.string,

{docsig-0.49.2 → docsig-0.50.0}/docsig/_module.py

@@ -10,6 +10,7 @@ import typing as _t
 from pathlib import Path as _Path
 
 import astroid as _ast
+import click as _click
 from astroid import AstroidSyntaxError as _AstroidSyntaxError
 
 from ._directives import Directive as _Directive
@@ -17,6 +18,7 @@ from ._directives import Directives as _Directives
 from ._message import Message as _Message
 from ._stub import Docstring as _Docstring
 from ._stub import Signature as _Signature
+from ._utils import pretty_print_error
 from ._utils import vprint as _vprint
 
 _FILE_INFO = "{path}: {msg}"
@@ -304,7 +306,7 @@ class Function(Parent):
         return self._directives
 
 
-class Modules(_t.List[Parent]):
+class Modules(_t.List[Parent]):  # pylint: disable=too-many-instance-attributes
     """Sequence of ``Module`` objects parsed from Python modules or str.
 
     Recursively collect Python files from within all dirs that exist
@@ -322,6 +324,7 @@ class Modules(_t.List[Parent]):
     :param check_class_constructor: Check the class constructor's
         docstring. Otherwise, expect the constructor's documentation to
         be on the class level docstring.
+    :param no_ansi: Disable ANSI output.
     :param verbose: increase output verbosity.
     """
 
@@ -352,22 +355,24 @@ class Modules(_t.List[Parent]):
                     check_class_constructor,
                 )
             )
+            _vprint(
+                _FILE_INFO.format(
+                    path=root or "stdin", msg="Parsing Python code successful"
+                ),
+                self._verbose,
+            )
         except (_AstroidSyntaxError, UnicodeDecodeError) as err:
+            msg = str(err).replace("\n", " ")
             if root is not None and root.name.endswith(".py"):
-                # keep raising errors for .py files as was done prior to
-                # this change
                 # pass by silently for files that do not end with .py,
-                # which were not checked at all prior (these may result
-                # in a 123 syntax error exit status in the future)
-                # with this there should be no breaking change, and
-                # files that are supposed to be python, files evident by
-                # their suffix, will continue to fail
-                raise err
+                # may result in a 123 syntax error exit status in the
+                # future
+                _click.echo(root)
+                pretty_print_error(type(err), msg, no_ansi=self._no_ansi)
+                self._retcode = 1
 
             _vprint(
-                _FILE_INFO.format(
-                    path=root or "stdin", msg=str(err).replace("\n", " ")
-                ),
+                _FILE_INFO.format(path=root or "stdin", msg=msg),
                 self._verbose,
             )
 
@@ -380,6 +385,7 @@ class Modules(_t.List[Parent]):
         ignore_args: bool = False,
         ignore_kwargs: bool = False,
         check_class_constructor: bool = False,
+        no_ansi: bool = False,
         verbose: bool = False,
     ) -> None:
         super().__init__()
@@ -388,7 +394,9 @@ class Modules(_t.List[Parent]):
         self._ignore_args = ignore_args
         self._ignore_kwargs = ignore_kwargs
         self.check_class_constructor = check_class_constructor
+        self._no_ansi = no_ansi
         self._verbose = verbose
+        self._retcode = 0
         if string is not None:
             self._add_module(
                 disable,
@@ -426,3 +434,8 @@ class Modules(_t.List[Parent]):
         if root.is_dir():
             for path in root.iterdir():
                 self._populate(path)
+
+    @property
+    def retcode(self) -> int:
+        """Return code to propagate to main."""
+        return self._retcode

{docsig-0.49.2 → docsig-0.50.0}/docsig/_utils.py

@@ -5,6 +5,8 @@ docsig._utils
 
 from __future__ import annotations as _
 
+import sys as _sys
+import typing as _t
 from difflib import SequenceMatcher as _SequenceMatcher
 
 import click as _click
@@ -31,3 +33,19 @@ def vprint(msg: str, verbose: bool = False) -> None:
     """
     if verbose:
         _click.echo(msg)
+
+
+def pretty_print_error(
+    exception_type: _t.Type[BaseException], msg: str, no_ansi: bool
+) -> None:
+    """Print user-friendly exception.
+
+    :param exception_type: Type of the exception.
+    :param msg: The exception message.
+    :param no_ansi: Whether to in ANSI escape codes.
+    """
+    _click.echo(
+        f"{_click.style(exception_type.__name__, fg='red', bold=True)}: {msg}",
+        file=_sys.stderr,
+        color=not no_ansi and _sys.stderr.isatty(),
+    )

{docsig-0.49.2 → docsig-0.50.0}/docsig/_version.py

@@ -8,4 +8,4 @@ Allows for access to the version internally without cyclic imports
 caused by accessing it through __init__.
 """
 
-__version__ = "0.49.2"
+__version__ = "0.50.0"

{docsig-0.49.2 → docsig-0.50.0}/pyproject.toml

@@ -16,16 +16,6 @@ exclude = '''
 '''
 line-length = 79
 
-[tool.constcheck]
-ignore_strings = [
-  "disable",
-  "red",
-  "store",
-  "store_true",
-  "targets",
-  "utf-8"
-]
-
 [tool.coverage.report]
 exclude_lines = [
   "@_t.overload"
@@ -73,7 +63,7 @@ maintainers = [
 name = "docsig"
 readme = "README.rst"
 repository = "https://github.com/jshwi/docsig"
-version = "0.49.2"
+version = "0.50.0"
 
 [tool.poetry.dependencies]
 Pygments = "^2.13.0"
@@ -104,7 +94,6 @@ docsig = "docsig.__main__:main"
 audit = [
   "about-tests",
   "commit-policy",
-  "const",
   "copyright-year",
   "docs",
   "files",

docsig-0.49.2/docsig/_hooks.py

@@ -1,22 +0,0 @@
-"""
-docsig._hooks
-=============
-"""
-
-import sys as _sys
-from os import environ as _e
-
-import click as _click
-
-
-def pretty_print_error(no_ansi: bool = False) -> None:
-    """Print user friendly commandline error if debug not enabled.
-
-    :param no_ansi: Disable ANSI output.
-    """
-    if _e.get("DOCSIG_DEBUG", None) != "1":
-        _sys.excepthook = lambda x, y, _: _click.echo(
-            f"{_click.style(x.__name__, fg='red', bold=True)}: {y}",
-            file=_sys.stderr,
-            color=not no_ansi and _sys.stderr.isatty(),
-        )