Skip to content

Comments

Align step names in release, super-linter, and mergefest workflows#10406

Closed
Copilot wants to merge 3 commits intomainfrom
copilot/align-step-names-in-lock-files
Closed

Align step names in release, super-linter, and mergefest workflows#10406
Copilot wants to merge 3 commits intomainfrom
copilot/align-step-names-in-lock-files

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

Three workflow files contained step names that deviated from established patterns: generic "Checkout" without target specification, "Checkout Code" using non-standard terminology, and "Setup git configuration" with lowercase "git".

Changes

  • release.md: CheckoutCheckout repository (2 instances)
  • super-linter.md: Checkout CodeCheckout repository
  • mergefest.md: Setup git configurationSetup Git configuration

Pattern compliance

  • Checkout steps now follow Checkout <target> convention (used 952× across codebase: "repository", "actions folder", "PR branch")
  • "Git" capitalized as proper noun, consistent with Configure Git credentials (193 instances)
Original prompt

This section details on the original issue you should resolve

<issue_title>[step-names] Align step names in release.lock.yml, super-linter.lock.yml, and mergefest.lock.yml</issue_title>
<issue_description>Found in: .github/workflows/release.lock.yml, .github/workflows/super-linter.lock.yml, .github/workflows/mergefest.lock.yml

Summary

Analysis of all 126 workflow lock files identified 3 step names with minor inconsistencies that should be aligned with established naming patterns. These are low-impact issues but fixing them will improve consistency across the codebase.

Issues Identified

1. [Medium Priority] Generic "Checkout" Step Name

Current step names:

  • .github/workflows/release.lock.yml:973 - "Checkout"
  • .github/workflows/release.lock.yml:1352 - "Checkout"

Issue:
These steps use the generic name "Checkout" which doesn't follow the established pattern Checkout <target>. The established pattern is used 758 times for "Checkout actions folder" and 194 times for "Checkout repository".

Suggested improvement:

  • Change both instances to "Checkout repository"

Pattern reference: The established pattern is Checkout <target> to provide clarity about what is being checked out.


2. [Medium Priority] "Checkout Code" Naming

Current step name:

  • .github/workflows/super-linter.lock.yml:1235 - "Checkout Code"

Issue:
Uses "Code" as the target, which is less specific than the standard "repository". All other checkout steps use either "Checkout repository" (194 instances), "Checkout actions folder" (758 instances), or "Checkout PR branch" (124 instances).

Suggested improvement:

  • Change "Checkout Code" to "Checkout repository"

Pattern reference: The established pattern uses specific, consistent target names: "repository", "actions folder", "PR branch".


3. [Medium Priority] Lowercase "git" in Setup Pattern

Current step name:

  • .github/workflows/mergefest.lock.yml:127 - "Setup git configuration"

Issue:
Uses lowercase "git" which is inconsistent with other Setup patterns that capitalize proper nouns. Examples of correct capitalization:

  • "Setup Node.js" (92 instances)
  • "Setup Go" (18 instances)
  • "Setup Python" (3 instances)
  • "Setup Python environment" (18 instances)
  • "Configure Git credentials" (193 instances) - Note: "Git" is capitalized here

Suggested improvement:

  • Change "Setup git configuration" to "Setup Git configuration"

Glossary reference: While not explicitly in the glossary, "Git" is a proper noun (the version control system) and should be capitalized consistently, as demonstrated by the 193 instances of "Configure Git credentials".


Positive Findings

The previous run on 2026-01-15 identified 4 instances of non-hyphenated "cache memory" and "repo memory" that violated the glossary. These have all been fixed!

All current instances now correctly use:

  • "cache-memory" (hyphenated) in technical contexts
  • "repo-memory" (hyphenated) in technical contexts

This demonstrates that the step name alignment process is working well.

Additional positive findings:

  • All 126 workflows follow imperative mood (no progressive verb forms like "Downloading")
  • MCP terminology correctly capitalized (MCP)
  • GitHub Copilot CLI uses full official name (160 instances)
  • Agentic terminology correct ("Generate agentic run info" - 126 instances)
  • Safe Outputs capitalization follows glossary (capitalized for feature, hyphenated lowercase for technical artifacts)
  • SBOM, Copilot, and other acronyms properly capitalized
  • Strong pattern consistency: "Setup Scripts" (763 instances), "Checkout actions folder" (758 instances)

Agentic Task Description

To fix these step names:

  1. Update source workflow files (.md files, not .lock.yml):

    • .github/workflows/release.md - Change two "Checkout" steps to "Checkout repository"
    • .github/workflows/super-linter.md - Change "Checkout Code" to "Checkout repository"
    • .github/workflows/mergefest.md - Change "Setup git configuration" to "Setup Git configuration"
  2. Recompile workflows - Run gh aw compile on the affected workflows to regenerate .lock.yml files

  3. Verify changes - Confirm the step names in the .lock.yml files match the expected names

  4. Test workflows - Ensure workflows still function correctly after the changes

Related Files

  • Source workflows: .github/workflows/release.md, .github/workflows/super-linter.md, .github/workflows/mergefest.md
  • Compiled workflows: .github/workflows/release.lock.yml, .github/workflows/super-linter.lock.yml, .github/workflows/mergefest.lock.yml
  • Project glossary: docs/src/content/docs/reference/glossary.md
  • Naming patterns: /tmp/gh-aw/cache-memory/step-name-alignment/naming-conventions.md

Priority

This issue is Medium Priority - these are minor inconsistencies that don't affect functionality but should be fixed to maintain consistency acros...


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 17, 2026 03:24
- Change "Checkout" to "Checkout repository" in release.md (2 instances)
- Change "Checkout Code" to "Checkout repository" in super-linter.md
- Change "Setup git configuration" to "Setup Git configuration" in mergefest.md
- Recompile affected workflow lock files

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Align step names in workflow lock files Align step names in release, super-linter, and mergefest workflows Jan 17, 2026
Copilot AI requested a review from mnkiefer January 17, 2026 03:36
@pelikhan pelikhan closed this Jan 17, 2026
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.

[step-names] Align step names in release.lock.yml, super-linter.lock.yml, and mergefest.lock.yml

3 participants