generated from jCloud/repository-template
Compare commits
15
Commits
3865db7919
..
v0.2.0
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
b2a8e54766
|
||
|
|
773294f3d9
|
||
|
|
200221e324
|
||
|
|
e8bcf4feb1
|
||
|
|
ff3bddd366
|
||
|
|
e94365faf8
|
||
|
|
95da53898c
|
||
|
|
4d2e63d560
|
||
|
|
2187590468
|
||
|
|
37308227d9
|
||
|
|
f796263c0e
|
||
|
|
896bfcc2dd
|
||
|
|
b2a7c9a0e3
|
||
|
|
ac72eb283c
|
||
|
|
21c5c20cb5
|
No files matched your search
@@ -39,8 +39,7 @@ jobs:
|
|||||||
|
|
||||||
- name: Generate documentation
|
- name: Generate documentation
|
||||||
run: |
|
run: |
|
||||||
rm -rf docs/reference
|
jcloud-docsgen python -T -f -o docs/reference ./src
|
||||||
jcloud-docsgen python -T -o docs/reference .
|
|
||||||
|
|
||||||
- name: Commit and push
|
- name: Commit and push
|
||||||
run: |
|
run: |
|
||||||
@@ -49,7 +48,7 @@ jobs:
|
|||||||
|
|
||||||
git add docs/reference
|
git add docs/reference
|
||||||
|
|
||||||
if git diff --chached --quit; then
|
if git diff --cached --quiet; then
|
||||||
echo "No changes"
|
echo "No changes"
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
## [Unreleased]
|
## [0.2.0] - 2026-09-28
|
||||||
|
|
||||||
### Added
|
### Added
|
||||||
|
|
||||||
|
|||||||
+1
-1
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
|
|||||||
|
|
||||||
[project]
|
[project]
|
||||||
name = "jcloud-docsgen"
|
name = "jcloud-docsgen"
|
||||||
version = "0.1.0"
|
version = "0.2.0"
|
||||||
description = "A tool that generates Markdown documentation for software project source code"
|
description = "A tool that generates Markdown documentation for software project source code"
|
||||||
dependencies = ["docstring-parser"]
|
dependencies = ["docstring-parser"]
|
||||||
requires-python = ">=3.10"
|
requires-python = ">=3.10"
|
||||||
|
|||||||
@@ -13,7 +13,7 @@
|
|||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
from . import cli
|
from . import cli
|
||||||
from .core import *
|
from .core import * # noqa: F403
|
||||||
from .core import __all__ as _core__all__
|
from .core import __all__ as _core__all__
|
||||||
|
|
||||||
__all__ = [
|
__all__ = [
|
||||||
|
|||||||
@@ -13,7 +13,7 @@
|
|||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
from . import namespaces
|
from . import namespaces
|
||||||
from ._core import *
|
from ._core import * # noqa: F403
|
||||||
from ._core import __all__ as _core__all__
|
from ._core import __all__ as _core__all__
|
||||||
|
|
||||||
__all__ = [
|
__all__ = [
|
||||||
|
|||||||
@@ -16,14 +16,13 @@ from __future__ import annotations
|
|||||||
|
|
||||||
import ast
|
import ast
|
||||||
import pathlib
|
import pathlib
|
||||||
|
import shutil
|
||||||
from collections.abc import Iterator
|
from collections.abc import Iterator
|
||||||
from dataclasses import dataclass
|
from dataclasses import dataclass
|
||||||
from typing import TypeAlias
|
from typing import TypeAlias
|
||||||
|
|
||||||
import docstring_parser
|
import docstring_parser
|
||||||
|
|
||||||
import shutil
|
|
||||||
|
|
||||||
from ...exceptions import (
|
from ...exceptions import (
|
||||||
DirectoryNotEmptyError,
|
DirectoryNotEmptyError,
|
||||||
InvalidNamespaceError,
|
InvalidNamespaceError,
|
||||||
@@ -250,9 +249,8 @@ def _markdown_table(
|
|||||||
if len(row) != header_len:
|
if len(row) != header_len:
|
||||||
if force_row_lengths:
|
if force_row_lengths:
|
||||||
raise ValueError(
|
raise ValueError(
|
||||||
f'expected {header_len} value{
|
f'expected {header_len} value'
|
||||||
"s" if header_len > 1 else ""
|
f'{"s" if header_len > 1 else ""}, got {len(row)}'
|
||||||
}, got {len(row)}',
|
|
||||||
)
|
)
|
||||||
else:
|
else:
|
||||||
row = _fit_row_length(header_len, row)
|
row = _fit_row_length(header_len, row)
|
||||||
@@ -278,9 +276,8 @@ def _markdown_table(
|
|||||||
if len(row) != header_len:
|
if len(row) != header_len:
|
||||||
if force_row_lengths:
|
if force_row_lengths:
|
||||||
raise ValueError(
|
raise ValueError(
|
||||||
f'expected {header_len} value{
|
f'expected {header_len} value'
|
||||||
"s" if header_len > 1 else ""
|
f'{"s" if header_len > 1 else ""}, got {len(row)}'
|
||||||
}, got {len(row)}',
|
|
||||||
)
|
)
|
||||||
else:
|
else:
|
||||||
row = _fit_row_length(header_len, row)
|
row = _fit_row_length(header_len, row)
|
||||||
@@ -602,11 +599,10 @@ class PythonDefinitionDocumentationGenerator:
|
|||||||
|
|
||||||
if _is_docstring(self.definition.doc) or not self.skip_empty_sections:
|
if _is_docstring(self.definition.doc) or not self.skip_empty_sections:
|
||||||
if self.allow_html:
|
if self.allow_html:
|
||||||
md += f'<details><summary><h{
|
md += (
|
||||||
str(self.level + 1)
|
f'<details><summary><h{str(self.level + 1)} style="displa'
|
||||||
} style="display:inline">Docstring</h{
|
f'y:inline">Docstring</h{str(self.level + 1)}></summary>'
|
||||||
str(self.level + 1)
|
)
|
||||||
}></summary>'
|
|
||||||
else:
|
else:
|
||||||
md += self.level * '#' + '# Docstring\n\n'
|
md += self.level * '#' + '# Docstring\n\n'
|
||||||
|
|
||||||
|
|||||||
@@ -28,9 +28,11 @@ class NamespaceError(ValueError):
|
|||||||
if not self.args:
|
if not self.args:
|
||||||
return ''
|
return ''
|
||||||
else:
|
else:
|
||||||
return f'{self.args[0]}{
|
return (
|
||||||
": " if self.namespace_identifier and self.args[0] else ""
|
self.args[0]
|
||||||
}{self.namespace_identifier if self.args[0] else ""}'
|
+ (': ' if self.namespace_identifier and self.args[0] else '')
|
||||||
|
+ (self.namespace_identifier if self.args[0] else '')
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
class InvalidNamespaceError(NamespaceError): ...
|
class InvalidNamespaceError(NamespaceError): ...
|
||||||
@@ -55,9 +57,11 @@ class PythonIdentifierError(ValueError):
|
|||||||
if not self.args:
|
if not self.args:
|
||||||
return ''
|
return ''
|
||||||
else:
|
else:
|
||||||
return f'{self.args[0]}{
|
return (
|
||||||
": " if self.identifier and self.args[0] else ""
|
self.args[0]
|
||||||
}{self.identifier if self.args[0] else ""}'
|
+ (': ' if self.identifier and self.args[0] else '')
|
||||||
|
+ (self.identifier if self.args[0] else '')
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
class InvalidPythonIdentifierError(PythonIdentifierError): ...
|
class InvalidPythonIdentifierError(PythonIdentifierError): ...
|
||||||
@@ -76,9 +80,11 @@ class PythonAnnotationError(ValueError):
|
|||||||
if not self.args:
|
if not self.args:
|
||||||
return ''
|
return ''
|
||||||
else:
|
else:
|
||||||
return f'{self.args[0]}{
|
return (
|
||||||
": " if self.annotation and self.args[0] else ""
|
self.args[0]
|
||||||
}{self.annotation if self.args[0] else ""}'
|
+ (': ' if self.annotation and self.args[0] else '')
|
||||||
|
+ (self.annotation if self.args[0] else '')
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
class InvalidPythonAnnotationError(PythonAnnotationError): ...
|
class InvalidPythonAnnotationError(PythonAnnotationError): ...
|
||||||
@@ -97,9 +103,11 @@ class PythonDefinitionError(ValueError):
|
|||||||
if not self.args:
|
if not self.args:
|
||||||
return ''
|
return ''
|
||||||
else:
|
else:
|
||||||
return f'{self.args[0]}{
|
return (
|
||||||
": " if self.definition and self.args[0] else ""
|
self.args[0]
|
||||||
}{self.definition if self.args[0] else ""}'
|
+ (': ' if self.definition and self.args[0] else '')
|
||||||
|
+ (self.definition if self.args[0] else '')
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
class PythonArgumentStructureError(PythonDefinitionError): ...
|
class PythonArgumentStructureError(PythonDefinitionError): ...
|
||||||
|
|||||||
@@ -20,7 +20,6 @@ import errno
|
|||||||
import os
|
import os
|
||||||
import pathlib
|
import pathlib
|
||||||
import types
|
import types
|
||||||
from typing import Union
|
|
||||||
|
|
||||||
try:
|
try:
|
||||||
from warnings import deprecated
|
from warnings import deprecated
|
||||||
@@ -154,10 +153,10 @@ def assert_that_is_instance(
|
|||||||
obj: object,
|
obj: object,
|
||||||
class_or_tuple: type | types.UnionType | tuple[type, ...],
|
class_or_tuple: type | types.UnionType | tuple[type, ...],
|
||||||
) -> None:
|
) -> None:
|
||||||
if not isinstance(class_or_tuple, Union[type, types.UnionType, tuple]):
|
if not isinstance(class_or_tuple, (type, types.UnionType, tuple)):
|
||||||
raise TypeError(
|
raise TypeError(
|
||||||
f"class_or_tuple: expected 'Union[type, types.UnionType, "
|
f"class_or_tuple: expected 'type | types.UnionType, tupl"
|
||||||
f"tuple[type, ...]]', 'got {type(class_or_tuple).__name__}'",
|
f"e[type, ...]', 'got {type(class_or_tuple).__name__}'",
|
||||||
)
|
)
|
||||||
if not isinstance(obj, class_or_tuple):
|
if not isinstance(obj, class_or_tuple):
|
||||||
if isinstance(class_or_tuple, (tuple, types.UnionType)):
|
if isinstance(class_or_tuple, (tuple, types.UnionType)):
|
||||||
@@ -175,9 +174,8 @@ def assert_that_is_instance(
|
|||||||
else:
|
else:
|
||||||
exception_message_expected = "'" + class_or_tuple.__name__ + "'"
|
exception_message_expected = "'" + class_or_tuple.__name__ + "'"
|
||||||
raise TypeError(
|
raise TypeError(
|
||||||
f"expected {exception_message_expected}, got '{
|
f'expected {exception_message_expected}'
|
||||||
type(obj).__name__
|
f", got '{type(obj).__name__}'"
|
||||||
}'",
|
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -13,11 +13,10 @@
|
|||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
import pathlib
|
import pathlib
|
||||||
|
import tempfile
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
|
|
||||||
import tempfile
|
|
||||||
|
|
||||||
from jcloud_docsgen.cli.cli import create_argument_parser
|
from jcloud_docsgen.cli.cli import create_argument_parser
|
||||||
from jcloud_docsgen.cli.cli import main as cli_main
|
from jcloud_docsgen.cli.cli import main as cli_main
|
||||||
from jcloud_docsgen.cli.languages.python import parse_sections_list
|
from jcloud_docsgen.cli.languages.python import parse_sections_list
|
||||||
|
|||||||
@@ -13,17 +13,16 @@
|
|||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
import pathlib
|
import pathlib
|
||||||
|
import tempfile
|
||||||
|
|
||||||
import pytest
|
import pytest
|
||||||
|
|
||||||
import tempfile
|
|
||||||
|
|
||||||
from src.jcloud_docsgen.exceptions import DirectoryNotEmptyError
|
|
||||||
from src.jcloud_docsgen.core.python import PythonDocumentationGenerator
|
from src.jcloud_docsgen.core.python import PythonDocumentationGenerator
|
||||||
from src.jcloud_docsgen.core.python.namespaces import (
|
from src.jcloud_docsgen.core.python.namespaces import (
|
||||||
PythonModuleNamespace,
|
PythonModuleNamespace,
|
||||||
PythonPackageNamespace,
|
PythonPackageNamespace,
|
||||||
)
|
)
|
||||||
|
from src.jcloud_docsgen.exceptions import DirectoryNotEmptyError
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.parametrize(
|
@pytest.mark.parametrize(
|
||||||
@@ -86,7 +85,9 @@ def test_PythonDocumentationGenerator_collect_modules(
|
|||||||
python_documentation_generator: PythonDocumentationGenerator,
|
python_documentation_generator: PythonDocumentationGenerator,
|
||||||
expected,
|
expected,
|
||||||
):
|
):
|
||||||
assert python_documentation_generator.namespace().sub_namespaces == expected
|
assert (
|
||||||
|
python_documentation_generator.namespace().sub_namespaces
|
||||||
|
) == expected
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.parametrize(
|
@pytest.mark.parametrize(
|
||||||
|
|||||||
+2
-2
@@ -125,12 +125,12 @@ Returns all users.
|
|||||||
##### Method signature
|
##### Method signature
|
||||||
|
|
||||||
```python
|
```python
|
||||||
list_all(self) -> List[User]
|
list_all(self) -> list[User]
|
||||||
```
|
```
|
||||||
|
|
||||||
##### Returns
|
##### Returns
|
||||||
|
|
||||||
Return type: `List[User]`
|
Return type: `list[User]`
|
||||||
|
|
||||||
##### Parameters
|
##### Parameters
|
||||||
|
|
||||||
|
|||||||
+1
-2
@@ -1,5 +1,4 @@
|
|||||||
from dataclasses import dataclass
|
from dataclasses import dataclass
|
||||||
from typing import Optional
|
|
||||||
|
|
||||||
|
|
||||||
@dataclass
|
@dataclass
|
||||||
@@ -11,4 +10,4 @@ class User:
|
|||||||
id: str
|
id: str
|
||||||
name: str
|
name: str
|
||||||
email: str
|
email: str
|
||||||
age: Optional[int] = None
|
age: int | None = None
|
||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
|
from .exceptions import InvalidEmailError, UserNotFoundError
|
||||||
from .models import User
|
from .models import User
|
||||||
from .storage import InMemoryUserStorage
|
from .storage import InMemoryUserStorage
|
||||||
from .exceptions import UserNotFoundError, InvalidEmailError
|
|
||||||
from .utils import is_valid_email, normalize_name
|
from .utils import is_valid_email, normalize_name
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
+2
-3
@@ -1,4 +1,3 @@
|
|||||||
from typing import Dict, List
|
|
||||||
from .models import User
|
from .models import User
|
||||||
|
|
||||||
|
|
||||||
@@ -9,7 +8,7 @@ class InMemoryUserStorage:
|
|||||||
"""
|
"""
|
||||||
|
|
||||||
def __init__(self):
|
def __init__(self):
|
||||||
self._users: Dict[str, User] = {}
|
self._users: dict[str, User] = {}
|
||||||
|
|
||||||
def add(self, user: User) -> None:
|
def add(self, user: User) -> None:
|
||||||
"""Adds a user."""
|
"""Adds a user."""
|
||||||
@@ -23,6 +22,6 @@ class InMemoryUserStorage:
|
|||||||
"""Deletes a user."""
|
"""Deletes a user."""
|
||||||
del self._users[user_id]
|
del self._users[user_id]
|
||||||
|
|
||||||
def list_all(self) -> List[User]:
|
def list_all(self) -> list[User]:
|
||||||
"""Returns all users."""
|
"""Returns all users."""
|
||||||
return list(self._users.values())
|
return list(self._users.values())
|
||||||
@@ -292,22 +292,25 @@ def test_PythonDocumentationGenerator_collect_modules(
|
|||||||
None,
|
None,
|
||||||
[
|
[
|
||||||
(
|
(
|
||||||
"""pytest.mark.parametrize('python_documentation_gener"""
|
"pytest.mark.parametrize('python_documentatio"
|
||||||
"""ator,expected', [(PythonDocumentationGenerator(path"""
|
"n_generator,expected', [(PythonDocumentation"
|
||||||
"""lib.Path('tests/unit/core/python/_core/test_project"""
|
"Generator(pathlib.Path('tests/unit/core/pyth"
|
||||||
"""_dirs/pdir_1'), pathlib.Path('tests/unit/core/pytho"""
|
"on/_core/test_project_dirs/pdir_1'), pathlib"
|
||||||
"""n/_core/test_project_dirs/pdir_1/docs')), []), (Pyt"""
|
".Path('tests/unit/core/python/_core/test_pro"
|
||||||
"""honDocumentationGenerator(pathlib.Path('tests/unit/"""
|
"ject_dirs/pdir_1/docs')), []), (PythonDocume"
|
||||||
"""core/python/_core/test_project_dirs/pdir_2'), pathl"""
|
"ntationGenerator(pathlib.Path('tests/unit/co"
|
||||||
"""ib.Path('tests/unit/core/python/_core/test_project_"""
|
"re/python/_core/test_project_dirs/pdir_2'), "
|
||||||
"""dirs/pdir_2/docs')), [PythonPackageNamespace('pkg',"""
|
"pathlib.Path('tests/unit/core/python/_core/t"
|
||||||
""" [PythonModuleNamespace('module')])]), (PythonDocum"""
|
"est_project_dirs/pdir_2/docs')), [PythonPack"
|
||||||
"""entationGenerator(pathlib.Path('tests/unit/core/pyt"""
|
"ageNamespace('pkg', [PythonModuleNamespace('"
|
||||||
"""hon/_core/test_project_dirs/pdir_3'), pathlib.Path("""
|
"module')])]), (PythonDocumentationGenerator("
|
||||||
"""'tests/unit/core/python/_core/test_project_dirs/pdi"""
|
"pathlib.Path('tests/unit/core/python/_core/t"
|
||||||
"""r_3/docs')), [PythonPackageNamespace('pkg', [Python"""
|
"est_project_dirs/pdir_3'), pathlib.Path('tes"
|
||||||
"""ModuleNamespace(\'module'), PythonPackageNamespace("""
|
'ts/unit/core/python/_core/test_project_dirs/'
|
||||||
"""\'pkg\', [PythonModuleNamespace(\'module2')])])])])"""
|
"pdir_3/docs')), [PythonPackageNamespace('pkg"
|
||||||
|
"', [PythonModuleNamespace('module'), Python"
|
||||||
|
"PackageNamespace('pkg', [PythonModuleNames"
|
||||||
|
"pace('module2')])])])])"
|
||||||
)
|
)
|
||||||
],
|
],
|
||||||
),
|
),
|
||||||
|
|||||||
Reference in new issue
Block a user