-
Notifications
You must be signed in to change notification settings - Fork 39
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
Use tox 4 to run test suite #3223
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3223 +/- ##
=======================================
Coverage 60.39% 60.39%
=======================================
Files 605 605
Lines 43683 43683
Branches 48 48
=======================================
Hits 26381 26381
Misses 17290 17290
Partials 12 12 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
d4c7569
to
413d550
Compare
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.
Finally :)
This was recommended in the tox docs: https://tox.wiki/en/latest/config.html#package
In the case of being in development mode and having set skipdist to true tox4 would not install NAV in the test environment
49e6493
to
63a3c30
Compare
Quality Gate passedIssues Measures |
Dependent on #3224 . Closes #2973.
Reference links:
https://tox.wiki/en/latest/upgrading.html
Escaping hashtags: https://tox.wiki/en/latest/upgrading.html#changed-ini-rules
Using
tox run
instead oftox
: https://tox.wiki/en/latest/upgrading.html#updating-usage-with-eSetting
package = editable
instead ofusedevelop = True
: https://tox.wiki/en/latest/config.html#packageConflict when using
skipdist = True
andusedevelop = True
: tox-dev/tox#2730