Skip to content

fix #109: Rename models/config_model to models/config#116

Merged
tomschr merged 2 commits intoopenSUSE:mainfrom
sushant-suse:issue#109
Dec 10, 2025
Merged

fix #109: Rename models/config_model to models/config#116
tomschr merged 2 commits intoopenSUSE:mainfrom
sushant-suse:issue#109

Conversation

@sushant-suse
Copy link
Copy Markdown
Collaborator

Linked Issue #109

Copy link
Copy Markdown
Contributor

@tomschr tomschr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. There is only two lines you forgot in src/docbuild/cli/cmd_cli.py:

diff --git i/src/docbuild/cli/cmd_cli.py w/src/docbuild/cli/cmd_cli.py
index ad98051..6299c5c 100644
--- i/src/docbuild/cli/cmd_cli.py
+++ w/src/docbuild/cli/cmd_cli.py
@@ -14,8 +14,8 @@ from typing import Any, cast
 from ..__about__ import __version__
 from ..config.app import replace_placeholders
 from ..config.load import handle_config
-from ..models.config_model.app import AppConfig
-from ..models.config_model.env import EnvConfig 
+from ..models.config.app import AppConfig
+from ..models.config.env import EnvConfig

If you apply these, then my tests succeeds. 😸

@sushant-suse sushant-suse marked this pull request as ready for review December 10, 2025 10:25
@tomschr tomschr merged commit db9b355 into openSUSE:main Dec 10, 2025
4 checks passed
@sushant-suse sushant-suse deleted the issue#109 branch December 10, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants