Skip to content

Conversation

@julien-lang
Copy link
Member

@julien-lang julien-lang commented Jan 22, 2026

@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

❌ Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.82%. Comparing base (4398a5a) to head (a028c3d).
⚠️ Report is 66 commits behind head on master.

Files with missing lines Patch % Lines
...thon/tk_framework_desktopserver/process_manager.py 0.00% 3 Missing ⚠️
...ython/tk_framework_desktopserver/shotgun/api_v1.py 0.00% 1 Missing ⚠️
...ython/tk_framework_desktopserver/shotgun/api_v2.py 50.00% 1 Missing ⚠️
...k_desktopserver/shotgun/scripts/execute_command.py 0.00% 1 Missing ⚠️
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     
Flag Coverage Δ
Linux 47.54% <0.00%> (?)
Python-3.10 65.82% <14.28%> (?)
Python-3.11 65.72% <14.28%> (?)
Python-3.9 65.79% <14.28%> (?)
Windows 63.80% <14.28%> (?)
macOS 64.34% <14.28%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@julien-lang julien-lang changed the title SG-38126 SG-38126 Update CI - Test results and code coverage Jan 23, 2026
@julien-lang julien-lang requested review from a team and Copilot January 23, 2026 00:05
@julien-lang julien-lang marked this pull request as ready for review January 23, 2026 00:05
Copy link

Copilot AI left a 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.

Copy link
Contributor

@carlos-villavicencio-adsk carlos-villavicencio-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@julien-lang julien-lang merged commit 6b38b9a into master Jan 28, 2026
21 of 22 checks passed
@julien-lang julien-lang deleted the ticket/SG-38126 branch January 28, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants