Skip to content

tests/sublibrary-ci: show Julia version in test job names#91

Merged
ChrisRackauckas merged 2 commits into
SciML:masterfrom
ChrisRackauckas-Claude:sublib-ci-job-version-name
Jun 14, 2026
Merged

tests/sublibrary-ci: show Julia version in test job names#91
ChrisRackauckas merged 2 commits into
SciML:masterfrom
ChrisRackauckas-Claude:sublib-ci-job-version-name

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Jun 14, 2026

Copy link
Copy Markdown

What

In the reusable sublibrary CI workflow (.github/workflows/sublibrary-project-tests.yml, the lib/* project-model sublibrary tests), reformat the test job's name: so the per-job display name surfaces the Julia version in a clearer, distinguishing form.

The job name already interpolated the version as a parenthesized lowercase suffix ((julia ${{ matrix.version }})). This changes it to a / Julia <version> form so the GitHub checks list reads e.g. lib/OrdinaryDiffEqTsit5 / Julia 1.10 vs lib/OrdinaryDiffEqTsit5 / Julia 1, instead of distinct-version jobs being hard to tell apart.

Before / After (the only line changed)

Before:

name: "${{ matrix.project }}${{ matrix.group != 'Core' && format(' [{0}]', matrix.group) || '' }} (julia ${{ matrix.version }})"

After:

name: "${{ matrix.project }}${{ matrix.group != 'Core' && format(' [{0}]', matrix.group) || '' }} / Julia ${{ matrix.version }}"

Details

  • Matrix variable carrying the Julia version is version, confirmed against both the workflow (matrix.version is also fed to julia-version:) and the matrix producer scripts/compute_affected_sublibraries.jl (--projects-matrix emits {"project":...,"group":...,"version":...,...}).
  • The existing project and group axes in the name are preserved; only the version suffix format changed.
  • Purely cosmetic: only the name: string changed — no functional/behavioral change.
  • actionlint clean on the edited file.

Not touched

  • sublibrary-downgrade.yml: its test job has no name: and its matrix only has a project axis — the Julia version is a single inputs.julia-version per run, not a per-job matrix distinguisher, so it is not the same multi-version-matrix situation. Left as-is.
  • Non-sublibrary tests.yml / grouped-tests.yml: out of scope.

NOTE: v1 must be retagged after merge for @v1 callers to pick this up.

Ignore until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code


Real fix added (tests.yml leaf job name)

For reusable-workflow calls, GitHub displays the called workflow's job name in the checks list, overriding the caller's richer name. So the visible "Tests - " entries come from the leaf reusable SciML/.github/.github/workflows/tests.yml@v1 tests job name — not from the callers (grouped-tests.yml / sublibrary-project-tests.yml). Adding the version only to the callers has no visible effect.

This PR now also edits .github/workflows/tests.yml to append the Julia version to that leaf job name:

Before:

name: "Tests${{ inputs.group != '' && format(' - {0}', inputs.group) || '' }}"

After:

name: "Tests${{ inputs.group != '' && format(' - {0}', inputs.group) || '' }} (Julia ${{ inputs.julia-version }})"

inputs.julia-version is a real on.workflow_call.inputs input of tests.yml (default "1", type string). The checks list will now read e.g. Tests - Extended (Julia 1.10) / (Julia 1) / (Julia pre) distinctly. actionlint clean.

The sublibrary-project-tests.yml caller-name change in this PR is cosmetic (it is the caller job name, not what GitHub shows for reusable calls); kept as-is.

ChrisRackauckas and others added 2 commits June 14, 2026 04:12
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…shown in checks)

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas-Claude ChrisRackauckas-Claude changed the title sublibrary-ci: include Julia version in each sub-job name tests/sublibrary-ci: show Julia version in test job names Jun 14, 2026
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 14, 2026 08:52
@ChrisRackauckas ChrisRackauckas merged commit 35a229a into SciML:master Jun 14, 2026
2 checks passed
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