Skip to content

Let GitHubActionsAttribute name the generated job with JobName - #697

Draft
ChrisonSimtian wants to merge 1 commit into
Fallout-build:developfrom
ChrisonSimtian:hotfix/v10.4.1-github-actions-job-name
Draft

ChrisonSimtian wants to merge 1 commit into
Fallout-build:developfrom
ChrisonSimtian:hotfix/v10.4.1-github-actions-job-name

Conversation

@ChrisonSimtian

@ChrisonSimtian ChrisonSimtian commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

What

Adds GitHubActionsAttribute.JobName. When set, it is used as both the generated job id and its name:. When unset, output is unchanged.

Why

The job name defaulted to the runner image. With RunsOnLabels (self-hosted runners) the image is only a placeholder, so every job was called e.g. windows-latest while running on [self-hosted]. That name is what appears in the PR checks list and what branch protection's required checks match on. Consumers had to subclass the attribute and override GetJobs to get around it.

Validation

  • Must be a valid job id: letters, digits, -, _, starting with a letter or _.
  • Only valid with a single image (a matrix would produce duplicate ids), asserted the same way as RunsOnLabels.

Tests

GitHubActionsJobNameSpecs: default naming unchanged (single image and matrix), job id and name: output when set, multi-image rejection, valid and invalid names.

Branching

Targets develop per CONTRIBUTING. The branch is cut from main, so it is a single commit that also applies cleanly to a 10.4.x hotfix.

@ChrisonSimtian
ChrisonSimtian changed the base branch from main to develop October 5, 2026 00:56
The job id and name defaulted to the runner image, so with RunsOnLabels
every job on a self-hosted runner was called after a placeholder image.
JobName overrides both. It must be a valid job id and requires a single
image; unset output is unchanged.
@ChrisonSimtian
ChrisonSimtian force-pushed the hotfix/v10.4.1-github-actions-job-name branch from eb1cb22 to f1c220f Compare October 5, 2026 00:57

This branch has not been deployed

No deployments
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.

1 participant