scikit-learn-intelex icon indicating copy to clipboard operation
scikit-learn-intelex copied to clipboard

Enable Coverity scan for C++

Open napetrov opened this issue 7 months ago • 4 comments

Description

Updates coverity job to instrument scikit build test scan - https://scan.coverity.com/projects/napetrov-daal4py?tab=overview

PR should start as a draft, then move to ready for review state after CI is passed and all applicable checkboxes are closed. This approach ensures that reviewers don't spend extra time asking for regular requirements.

You can remove a checkbox as not applicable only if it doesn't relate to this PR in any way. For example, PR with docs update doesn't require checkboxes for performance while PR with any change in actual code should have checkboxes and justify how this code change is expected to affect performance (or justification should be self-evident).

Checklist to comply with before moving PR from draft:

PR completeness and readability

  • [ ] I have reviewed my changes thoroughly before submitting this pull request.

Testing

  • [ ] I have run it locally and tested the changes extensively.

napetrov avatar Jun 11 '25 05:06 napetrov

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Flag Coverage Δ
azure 79.93% <ø> (+0.02%) :arrow_up:
github 71.62% <ø> (+<0.01%) :arrow_up:

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

see 4 files with indirect coverage changes

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

codecov[bot] avatar Jun 11 '25 06:06 codecov[bot]

/azp run Nightly

icfaust avatar Jun 11 '25 07:06 icfaust

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Jun 11 '25 07:06 azure-pipelines[bot]

Coverity job will not be passing secret to forks. so i do have test project - https://scan.coverity.com/projects/napetrov-daal4py?tab=overview

napetrov avatar Jun 11 '25 18:06 napetrov