-
Notifications
You must be signed in to change notification settings - Fork 14
Document ordering of test modules #176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
mrbean-bremen
commented
May 2, 2025
- see [Label] Allow specifying full directories/files as order targets #51
@jenstroeger - can you please have a look? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #176 +/- ##
=======================================
Coverage 98.50% 98.50%
=======================================
Files 5 5
Lines 601 601
Branches 131 131
=======================================
Hits 592 592
Misses 2 2
Partials 7 7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, good to see this documented 👍🏼
Can you also mention what happens to modules without marker mixed with modules with order marker? And I suspect -1
still puts a module at the end? Other than that let me just approve…
a7f9df2
to
d0235e7
Compare
@jenstroeger - Thanks for the review! I'll leave the PR open for a bit in case you want to have a final look. |
All good, thank you 🤓 |