Closed leandrodamascena closed 2 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.43%. Comparing base (
e14e768
) to head (d198233
). Report is 599 commits behind head on develop.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issue number: #4519
Additional issue: #4520
Summary
Changes
This PR introduces the use of Nox for testing specific dependencies in each feature we have. This is essential to prevent regressions.
Using Nox allows us to test the behavior of required/optional dependencies during our tests. This way, we can isolate the feature being tested from the actual implementation of its dependencies, making the tests more reliable, consistent, and not adding anything that require extras libraries. With this we can catch potential regressions early in the development process, before they make it to the prod.
User experience
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Is this a breaking change?
**RFC issue number**: Checklist: * [ ] Migration process documented * [ ] Implement warnings (if it can live side by side)Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.