refactor: convert LineZone docstring example to doctest#2207
refactor: convert LineZone docstring example to doctest#2207Borda merged 4 commits intoroboflow:developfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #2207 +/- ##
=======================================
Coverage 77% 77%
=======================================
Files 66 66
Lines 8189 8189
=======================================
Hits 6303 6303
Misses 1886 1886 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR updates the LineZone class documentation by converting its docstring example into a doctest-friendly interactive session, supporting the repo-wide goal of continuously validating documentation examples.
Changes:
- Replaced the previous static Markdown example in
LineZone’s docstring with a minimal>>>-style doctest example.
|
@Borda I've updated the
Regarding the other files (like |
For these, we can create an integration group in pyproject.toml, and in the docstrings, we can have a conditional skip if a dependency is missing... and have one simple CI job which would install all the integration group 🦝 See section "Skip Unconditionally" in the https://pypi.org/project/pytest-doctestplus/ where this would be inside the condition |
Before submitting
Description
i converted static docstring examples to doctest format for the LineZones file
Type of Change
Motivation and Context
Part of #2106
Additional note
It's actually my first PR on the project. I have only changed one file, but if the change happens to be correct, I could improve this PR by doing the conversion on more files.