Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #238 +/- ##
=======================================
Coverage 99.27% 99.27%
=======================================
Files 8 8
Lines 971 971
=======================================
Hits 964 964
Misses 7 7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@seproDev |
|
@fabiocaccamo The test does currently fail under Windows. The issue is that |
|
@seproDev could you please add a failing unit test to this PR? |
|
@fabiocaccamo |
|
@seproDev I see it, thanks for explaining. |
|
@seproDev I fixed the tests in a simpler way, thank you very much for catching these issues. |
name: Pull request
about: Submit a pull request for this project
assignees: fabiocaccamo
Describe your changes
Fixes the test on Windows. Windows uses
\as the separator.Related issue
?
Checklist before requesting a review