mirror of
https://github.com/NVIDIA/Model-Optimizer.git
synced 2026-10-02 03:14:52 +08:00
Keval MorabiaandClaude Opus 5
99116c3fb9
### What does this PR do? Type of change: CI/CD maintenance #2102 and #2103 bumped every action this repo references *directly*, but Node 20 annotations still appear — e.g. [this run on #2103](https://github.com/NVIDIA/Model-Optimizer/actions/runs/31170982088?pr=2103): > The following actions target Node.js 20 but are being forced to run on Node.js 24: `actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea` We never reference `github-script`. It comes in transitively through `codecov/codecov-action`: | codecov-action | bundled github-script | runtime | |---|---|---| | `@v5` (= v5.5.5) | `60a0d830…` | **node20** | | `@v6.0.2` / `@v7.0.0` | `ed597411…` | node24 | Bumps all four call sites (`unit_tests`, `gpu_tests`, `regression_tests`, `_example_tests_runner`) to `@v7`. Checked: v6's notes call out node24 support as the only breaking aspect — the same pattern as the bumps in #2102 — and our runners report 2.336.0. The inputs used here (`token`, `files`, `flags`, `fail_ci_if_error`, `verbose`) all still exist in v7. A scan of every referenced action, direct and one level transitive, now finds no node20 runtimes left. ### Testing Coverage upload runs in every unit, gpu, regression and example job, so CI exercises this broadly. Worth checking that coverage still lands in Codecov rather than only that the step is green — `fail_ci_if_error: false` means an upload failure would not turn the job red. ### Before your PR is "*Ready for review*" - Is this change backward compatible?: ✅ - If you copied code from any other sources or added a new PIP dependency, did you follow guidance in `CONTRIBUTING.md`: N/A - Did you write any new necessary tests?: N/A — CI configuration - Did you update [Changelog](https://github.com/NVIDIA/Model-Optimizer/blob/main/CHANGELOG.rst)?: N/A - Did you get Claude approval on this PR?: ❌ — not yet run <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated automated test workflows to use the latest coverage reporting action. * Improved compatibility and reliability of coverage report uploads across example, GPU, regression, and unit tests. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Keval Morabia <28916987+kevalmorabia97@users.noreply.github.com> Co-authored-by: Claude Opus 5 <noreply@anthropic.com>