Skip to content

Conversation

@julien-lang
Copy link
Member

@julien-langjulien-lang commented Jan 12, 2026

Changes align with shotgunsoftware/sg-jira-bridge#103 and shotgunsoftware/tk-ci-tools#73.

Fixups

  • Fix a code coverage issue regarding the certifi third party library
    • The certifi library was included in the coverage result despites being listed in the .coveragerc ignore list because of a typo
  • Fix Coverage badge in readme
    • was targetinng old coverall.io service that we don'T use since we move to Azure Pipelines
    • Now targets codecov.io

Pytests

Consolidate test structure around pytest.

  • Remove useless driver script. It's just as easy to launch pytest.
  • Remove outdated information about nose
  • Rename the PIP requirement files from ci_requirements.txt to requirements.txt

Azure Pipeline CI Improvements

  • Define job parameter in the "formal" form without a default value to make sure CI fails if the parameters are not provided
  • Simplify the code coverage upload mechanism to codecov.io
  • Drop pytest-azurepipeline
    • This package was prviously used to automatically upload test report and code coverage to Azure Pipelines which is very convenient.
    • Unfortunately, the package does not seem to be maintained anymore. And it is using outdated versions the PublishTestResults and PublishCodeCoverageResults Azure Pipeline tasks.

Copy link
Contributor

CopilotAI 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 code coverage tooling from Coveralls to Codecov, correcting configuration issues and removing deprecated dependencies.

Changes:

  • Migrated coverage reporting from Coveralls to Codecov
  • Replaced pytest-coverage with pytest-cov and removed the coveralls package
  • Fixed typo in coverage configuration

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

FileDescription
tests/ci_requirements.txtRemoved coveralls dependency and replaced pytest-coverage with pytest-cov
README.mdUpdated coverage badge to point to Codecov instead of Coveralls
.coveragercFixed typo in certifi directory name and updated configuration comment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecovbot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.66%. Comparing base (a5c32bc) to head (9ec4bbe).

Additional details and impacted files
@@ Coverage Diff @@## master #439 +/- ## ========================================== + Coverage 77.07% 78.66% +1.59%  ========================================== Files 10 7 -3 Lines 1880 1842 -38 ========================================== Hits 1449 1449 + Misses 431 393 -38 
FlagCoverage Δ
Linux78.66% <ø> (?)
Python-3.1078.61% <ø> (?)
Python-3.1178.61% <ø> (?)
Python-3.1378.61% <ø> (?)
Python-3.978.59% <ø> (?)
Windows78.75% <ø> (?)
macOS78.66% <ø> (?)

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-langjulien-langforce-pushed the ticket/SG-38126-review-and-fixup-code-coverage branch from 6b9c5b7 to 4041cd9CompareJanuary 13, 2026 19:29
@julien-langjulien-lang changed the base branch from master to ticket/SG-40980-python-3-13-ciJanuary 13, 2026 19:30
Base automatically changed from ticket/SG-40980-python-3-13-ci to masterJanuary 14, 2026 17:10
@julien-langjulien-langforce-pushed the ticket/SG-38126-review-and-fixup-code-coverage branch from 4041cd9 to 96c215bCompareJanuary 14, 2026 18:39
Copy link
Contributor

CopilotAI 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

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@julien-langjulien-lang changed the title SG-38126 Fixups regarding Code CoverageSG-38126 Review and Fixups CI Tests and Code CoverageJan 19, 2026
@julien-langjulien-lang marked this pull request as ready for review January 21, 2026 00:55
@julien-langjulien-lang requested a review from a teamJanuary 22, 2026 21:17
Sign up for freeto 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

@julien-lang@eduardoChaucaGallegos