Skip to content

CI: Update final file to use hashes. - #87

Merged
zssherman merged 1 commit into
ARM-DOE:mainfrom
zssherman:final_change
Oct 15, 2026
Merged

zssherman merged 1 commit into
ARM-DOE:mainfrom
zssherman:final_change

Conversation

@zssherman

Copy link
Copy Markdown
Contributor

PR tags (required)

Select all that apply.

  • docs-only
  • tests-only
  • tiny-change (<5 lines, single class/file)
  • code-change
  • scientific/algorithmic
  • design/architecture
  • GUI

What changed

2-4 bullets in plain language.

Why

1-2 lines on the user/problem impact.

Issues closed

List issue references this PR closes (for example: Closes #123, Fixes #456).

Target version (required)

Aiming for: vX.Y.Z / next patch / unreleased

Validation

Mark each item as done or N/A. Add one short evidence line for each item.

  • I ran ruff

    • Evidence (command + pass/fail):
    • N/A reason (if not run):
  • I ran import-linter

    • Evidence (command + pass/fail):
    • N/A reason (if not run):
  • I ran tests

    • Evidence (command + pass/fail):
    • N/A reason (if not run):

AI usage & manual verification

If AI tools were used, complete this section.

  • AI used: yes/no
  • Model/tooling:
  • Tasks delegated to AI:
  • Manual verification performed:

Reviewer guide

Where should review start, and what are the key files/areas?

  • Entry point:
  • Key files/areas (1-3):
  • Expected review time:

Impact note (optional): breaking behavior, scientific output shift, or user-visible behavior.

@codecov-commenter

codecov-commenter commented Oct 15, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.62%. Comparing base (4d927cb) to head (00848df).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   89.62%   89.62%           
=======================================
  Files         113      113           
  Lines        6575     6575           
  Branches      827      827           
=======================================
  Hits         5893     5893           
  Misses        476      476           
  Partials      206      206           
Flag Coverage Δ
unittests 89.56% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@zssherman
zssherman merged commit c7d5c72 into ARM-DOE:main Oct 15, 2026
10 checks passed
@zssherman
zssherman deleted the final_change branch October 15, 2026 19:22
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