Skip to content

[WIP] codecov testing #891

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from
Open

[WIP] codecov testing #891

wants to merge 16 commits into from

Conversation

michaelmckinsey1
Copy link
Collaborator

Description

  • Replace with: A short description of the change, including motivation and context.
  • Replace with: A list of any dependencies.
  • Replace with: Link(s) to relevant issue(s)
  • Complete the checklist for a relevant section(s) below
  • Delete sections below that are not relevant to this PR

Adding/modifying a system (docs: Adding a System)

  • Add/modify systems/system_name/system.py file
  • Add/modify a dry run unit test for system_name in .github/workflows/run.yml
  • Add/modify systems/all_hardware_descriptions/hardware_name/hardware_description.yaml which will appear in the docs catalogue

Adding/modifying a benchmark (docs: Adding a Benchmark)

  • If modifying the source code of a benchmark: create, self-assign, and link here a follow up issue with a link to the PR in the benchmark repo.
  • If package.py upstreamed to Spack is insufficient, add/modify repo/benchmark_name/package.py plus: create, self-assign, and link here a follow up issue with a link to the PR in the Spack repo.
  • If application.py upstreamed to Ramble is insufficient, add/modify repo/benchmark_name/application.py plus: create, self-assign, and link here a follow up issue with a link to the PR in the Ramble repo.
  • Tags in Ramble's application.py or in repo/benchmark_name/application.py will appear in the docs catalogue
  • Add/modify an experiments/benchmark_name/experiment.py to define a single node and multi-node experiments
  • Add/modify a dry run unit test in .github/workflows/run.yml

Adding/modifying core functionality, CI, or documentation:

  • Update docs
  • Update .github/workflows and .gitlab/tests unit tests (if needed)

@github-actions github-actions bot added feature New feature or request ci CI, unit tests, GitHub actions labels Jul 11, 2025
@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Jul 11, 2025
@michaelmckinsey1 michaelmckinsey1 marked this pull request as draft July 11, 2025 22:49
@michaelmckinsey1 michaelmckinsey1 marked this pull request as ready for review July 11, 2025 22:49
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.99%. Comparing base (5ac8be9) to head (e50af43).

Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #891       +/-   ##
============================================
+ Coverage    41.48%   61.99%   +20.50%     
============================================
  Files           22       43       +21     
  Lines         1509     2952     +1443     
  Branches       213      213               
============================================
+ Hits           626     1830     +1204     
- Misses         838     1114      +276     
+ Partials        45        8       -37     

see 32 files with indirect coverage changes

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

@michaelmckinsey1 michaelmckinsey1 self-assigned this Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CI, unit tests, GitHub actions dependencies Pull requests that update a dependency file feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants