-
Notifications
You must be signed in to change notification settings - Fork 9
SG-38126 Update CI - Test results and code coverage #308
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
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
carlos-villavicencio-adsk
left a comment
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.
lgtm
Uh oh!
There was an error while loading. Please reload this page.