Skip to content

Conversation

@julien-lang
Copy link
Member

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

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.

@julien-lang julien-lang requested a review from Copilot January 12, 2026 22:33
Copy link
Contributor

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 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.

File Description
tests/ci_requirements.txt Removed coveralls dependency and replaced pytest-coverage with pytest-cov
README.md Updated coverage badge to point to Codecov instead of Coveralls
.coveragerc Fixed 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

codecov bot 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 (eb40e12).

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     
Flag Coverage Δ
Linux 78.66% <ø> (?)
Python-3.10 78.61% <ø> (?)
Python-3.11 78.61% <ø> (?)
Python-3.13 78.61% <ø> (?)
Python-3.9 78.59% <ø> (?)
Windows 78.75% <ø> (?)
macOS 78.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-lang julien-lang force-pushed the ticket/SG-38126-review-and-fixup-code-coverage branch from 6b9c5b7 to 4041cd9 Compare January 13, 2026 19:29
@julien-lang julien-lang changed the base branch from master to ticket/SG-40980-python-3-13-ci January 13, 2026 19:30
Base automatically changed from ticket/SG-40980-python-3-13-ci to master January 14, 2026 17:10
@julien-lang julien-lang force-pushed the ticket/SG-38126-review-and-fixup-code-coverage branch from 4041cd9 to 96c215b Compare January 14, 2026 18:39
@julien-lang julien-lang requested a review from Copilot January 14, 2026 18:53
Copy link
Contributor

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

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-lang julien-lang changed the title SG-38126 Fixups regarding Code Coverage SG-38126 Review and Fixups CI Tests and Code Coverage Jan 19, 2026
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.

2 participants