Skip to content

fix(ci): daily workflow job names#2836

Merged
richard-ramos merged 1 commit into
masterfrom
fix/daily-name
Jul 15, 2026
Merged

fix(ci): daily workflow job names#2836
richard-ramos merged 1 commit into
masterfrom
fix/daily-name

Conversation

@richard-ramos

@richard-ramos richard-ramos commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

Refactors daily_test_all_latest_deps.yml so the PR label gate runs on a static wrapper job, while the actual test matrix lives in daily_test_all_latest_deps_matrix.yml.

This avoids skipped PR runs showing raw ${{ matrix... }} expressions, while preserving detailed matrix job names when the daily workflow actually runs.

Affected Areas

  • Build / Tooling
    • Daily GitHub Actions workflow structure for latest-dependency test_all runs.

Risk Assessment

Low risk. The workflow logic is unchanged operationally; the existing cache cleanup, dependency resolution, and test matrix were moved into a reusable workflow invoked by the gated daily wrapper.

@richard-ramos
richard-ramos marked this pull request as ready for review July 15, 2026 18:17
@richard-ramos
richard-ramos requested review from a team, gmelodie and vladopajic July 15, 2026 18:17
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.23%. Comparing base (da61305) to head (6259ce0).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2836      +/-   ##
==========================================
- Coverage   81.24%   81.23%   -0.01%     
==========================================
  Files         170      170              
  Lines       30741    30741              
  Branches       12       12              
==========================================
- Hits        24976    24973       -3     
- Misses       5765     5768       +3     

see 4 files with indirect coverage changes

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

@richard-ramos
richard-ramos enabled auto-merge July 15, 2026 19:26

@jakubgs jakubgs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Appreciate it.

@richard-ramos
richard-ramos added this pull request to the merge queue Jul 15, 2026
@github-project-automation github-project-automation Bot moved this from new to In Progress in nim-libp2p Jul 15, 2026
Merged via the queue into master with commit 13473bc Jul 15, 2026
60 of 61 checks passed
@richard-ramos
richard-ramos deleted the fix/daily-name branch July 15, 2026 20:59
@github-project-automation github-project-automation Bot moved this from In Progress to done in nim-libp2p Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

5 participants