Skip to content

Clarify Step 5 VS Code branch merge and delete instructions#31

Open
0xhaaz wants to merge 1 commit into
skills:mainfrom
0xhaaz:main
Open

Clarify Step 5 VS Code branch merge and delete instructions#31
0xhaaz wants to merge 1 commit into
skills:mainfrom
0xhaaz:main

Conversation

@0xhaaz

@0xhaaz 0xhaaz commented Jun 7, 2026

Copy link
Copy Markdown

Summary

Updates Step 5, Activity 2 to make the VS Code branch workflow clearer. The original instructions did not specify which three-dot menu to use for merging and deleting the add-level-counter branch.

Changes

  • Clarified which Source Control sidebar menu to use for Merge Branch and Delete Branch.
  • Added a screenshot that highlights the correct menu location.
  • Moved the add-level-counter branch name before its screenshot example.

Closes: #30

Task list

  • For workflow changes, I have verified the Actions workflows function as expected.
  • For content changes, I have reviewed the style guide.

Copilot AI review requested due to automatic review settings June 7, 2026 03:44

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

Note

Copilot was unable to run its full agentic suite in this review.

Updates the step-by-step Git workflow documentation to reflect a revised UI flow and refresh associated screenshots.

Changes:

  • Repositioned an existing screenshot to align with the branch-name instruction.
  • Updated merge/delete branch instructions to a new “REPOSITORIES” three-dot menu flow.
  • Switched one screenshot reference from a GitHub-hosted attachment to a repo-relative asset path.

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

Comment thread .github/steps/5-step.md Outdated
Comment thread .github/steps/5-step.md Outdated
Comment thread .github/steps/5-step.md
Comment thread .github/steps/5-step.md Outdated
Comment thread .github/steps/5-step.md Outdated
Comment thread .github/steps/5-step.md Outdated

<img width="350px" src="https://github.com/user-attachments/assets/aa35bdc4-9a90-4c62-a43c-9de9838d15a6"/><br/>

Notice that it performed a normal Fast Forward-style merge.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

issue here

Updated Activity 2 to identify the correct three-dot menu location in the VS Code Source Control sidebar when merging and deleting a branch. Added an updated screenshot showing the correct menu location.

Also moved the add-level-counter branch name before its screenshot example so learners see the required input before the visual example.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📋 Backlog

Development

Successfully merging this pull request may close these issues.

[Bug] Ambiguous VS Code branch menu instructions and screenshot order in Step 5, Activity 2

4 participants