Skip to content

Fix/remove doctests#79

Merged
maldwg merged 3 commits intomainfrom
fix/remove-doctests
May 7, 2025
Merged

Fix/remove doctests#79
maldwg merged 3 commits intomainfrom
fix/remove-doctests

Conversation

@maldwg
Copy link
Collaborator

@maldwg maldwg commented May 7, 2025

Resolves #51 by deleting the only doctest I could find as it was already covered by basic tests

@stefanDeveloper stefanDeveloper self-requested a review May 7, 2025 07:46
@stefanDeveloper stefanDeveloper added the enhancement New feature or request label May 7, 2025
@github-actions
Copy link

github-actions bot commented May 7, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1439 1430 99% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/base/utils.py 100% 🟢
TOTAL 100% 🟢

updated for commit: 7737dc4 by action🐍

Copy link
Owner

@stefanDeveloper stefanDeveloper left a comment

Choose a reason for hiding this comment

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

lgtm

@maldwg maldwg merged commit a4cf83c into main May 7, 2025
37 of 38 checks passed
@stefanDeveloper stefanDeveloper deleted the fix/remove-doctests branch September 23, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Doctests are not executed when committing

2 participants