SG-38126 Update CI - Test results and code coverage#308
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #308 +/- ##
===========================================
+ Coverage 41.02% 65.82% +24.80%
===========================================
Files 284 24 -260
Lines 43483 1870 -41613
===========================================
- Hits 17840 1231 -16609
+ Misses 25643 639 -25004
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:
|
There was a problem hiding this comment.
Pull request overview
This PR updates the CI configuration to improve test result handling and code coverage reporting. It implements changes from a CI tools update, ensuring separate test results and coverage files for integration tests, and updates pre-commit dependencies.
Changes:
- Updated CI to generate separate test results and coverage files for interpreter integration tests
- Removed deprecated test execution code from integration test file
- Updated pre-commit hook versions (pre-commit-hooks and black)
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| azure-pipelines.yml | Updated CI reference and configured separate test results/coverage for interpreter tests |
| tests/integration_tests/interpreters.py | Removed deprecated standalone test execution code |
| .pre-commit-config.yaml | Updated pre-commit-hooks and black versions |
| tests/fixtures/config/bundles/test_app/app.py | Added blank line for formatting consistency |
| python/tk_framework_desktopserver/shotgun/scripts/execute_command.py | Removed unnecessary parentheses from tuple unpacking |
| python/tk_framework_desktopserver/shotgun/api_v2.py | Removed unnecessary parentheses from tuple unpacking |
| python/tk_framework_desktopserver/shotgun/api_v1.py | Removed unnecessary parentheses from tuple unpacking |
| python/tk_framework_desktopserver/process_manager.py | Removed unnecessary parentheses from tuple unpacking |
| python/tk_framework_desktopserver/init.py | Removed extraneous blank line |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.