Skip to content

Rename checkout steps to switch in action.yml#6

Merged
mmathieum merged 2 commits into
masterfrom
mmathieum-patch-1
Apr 17, 2026
Merged

Rename checkout steps to switch in action.yml#6
mmathieum merged 2 commits into
masterfrom
mmathieum-patch-1

Conversation

@mmathieum

Copy link
Copy Markdown
Member

No description provided.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the GitHub Action setup to replace git checkout with git switch and renames associated step names and log messages for consistency. A review comment suggests updating an overlooked log message on line 48 to use 'Switching' instead of 'Checking out' to align with the rest of the changes.

Comment thread .github/actions/setup/action.yml

Copilot AI 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.

Pull request overview

Renames GitHub Actions composite setup step labels (and one log line) from “checkout” terminology to “switch”, and updates the submodule SHA command to use git switch --detach, improving consistency with modern Git commands.

Changes:

  • Renamed composite-action step names from “check out/checkout” to “switch”.
  • Updated echo output for branch switching steps to say “Switching”.
  • Replaced git checkout $MT_SHA with git switch --detach $MT_SHA when checking out a workflow SHA in a submodule context.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/actions/setup/action.yml
Comment thread .github/actions/setup/action.yml Outdated
@mmathieum mmathieum merged commit 0415901 into master Apr 17, 2026
@mmathieum mmathieum deleted the mmathieum-patch-1 branch April 17, 2026 18:35
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