Skip to content

Updates to CodeCov configuration#3491

Merged
blnicho merged 8 commits intoPyomo:mainfrom
jsiirola:codecov-config-update
Mar 4, 2025
Merged

Updates to CodeCov configuration#3491
blnicho merged 8 commits intoPyomo:mainfrom
jsiirola:codecov-config-update

Conversation

@jsiirola
Copy link
Copy Markdown
Member

@jsiirola jsiirola commented Feb 24, 2025

Fixes # .

Summary/Motivation:

This resolves some issues with our CodeCov configuration:

  • Allow codecov to generates reports even when the CI fails. This should help ensure that we get coverage information more often (even in the presence of intermittent infrastructure failures)
  • prevent uploads with multiple flags (codecov is issuing warnings for reports with multiple flags)
  • simplify the job name when reporting from Jenkins
  • rename codecov.yml -> .codecov.yml to clean up the project directory

Changes proposed in this PR:

  • (see above)

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@blnicho
Copy link
Copy Markdown
Member

blnicho commented Feb 25, 2025 •
edited
Loading

@jsiirola I brought this up at the dev call last week but I noticed that codecov has some new test analytics features: https://docs.codecov.com/docs/test-analytics

Since this PR is already making changes to the codecov configuration, should we go ahead and enable these test analytics while we're at it?

**EDIT: We discussed this during the dev call and decided that the codecode test analytics would require significant changes beyond what's included in this PR so we will move forward with this PR as-is.

@jsiirola
Copy link
Copy Markdown
Member Author

jsiirola commented Feb 27, 2025

I did a quick look at the test analytics feature and it looks like it would require a somewhat significant restructuring of the GHA drivers to have each job always upload both coverage and the test result. Right now we bundle the codecov results into only 5 uploads because a while back CodeCov was having trouble when we uploaded all our job results individually.

Given the effort, I think I would argue for doing it in a separate PR.

@blnicho blnicho merged commit 515d5d1 into Pyomo:main Mar 4, 2025
33 of 34 checks passed
@jsiirola jsiirola deleted the codecov-config-update branch March 24, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blnicho blnicho blnicho approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jsiirola @blnicho @emma58