Skip to content

ci: add names to changed-files output steps for readability - #13387

Open
MGPOCKY wants to merge 2 commits into
EbookFoundation:mainfrom
MGPOCKY:fix/ci-workflow-step-names
Open

ci: add names to changed-files output steps for readability#13387
MGPOCKY wants to merge 2 commits into
EbookFoundation:mainfrom
MGPOCKY:fix/ci-workflow-step-names

Conversation

@MGPOCKY

@MGPOCKY MGPOCKY commented Aug 1, 2026

Copy link
Copy Markdown

Problem

In .github/workflows/check-urls.yml, the set-files step had no name:, so a long run block shows up as a generic label in the Actions UI.

Changes

Semantics are unchanged: same id: set-files, same GITHUB_OUTPUT keys (files, files-len), and the same job-level output wiring.

  • Add name: Set changed-files outputs to the existing set-files step
  • Add name: Build check-urls matrix to the existing set-matrix step

Test plan

  • Confirm job outputs still resolve steps.set-files.outputs.files and steps.set-files.outputs.files-len
  • Open a PR that touches a .md / .yml file and confirm check-urls still runs

MGPOCKY and others added 2 commits August 1, 2026 04:58
Only add names for readability; do not split outputs across step ids.

Co-authored-by: Cursor <cursoragent@cursor.com>
@MGPOCKY MGPOCKY changed the title ci: split changed-files output steps for readability ci: add names to changed-files output steps for readability Aug 1, 2026
@MGPOCKY
MGPOCKY marked this pull request as ready for review August 1, 2026 05:01

@daltino daltino left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for working on this! The changes look neat and well-structured.

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