Update steps activities - #9
Conversation
…re issue template
…orkflow, and summarize exercise completion
…checks for calculator functions and issue criteria
Fix [!NOTE] indenting
Fix [!NOTE] and [!TIP] indenting
Disable AI features in chat settings.
Disable AI features in chat settings.
- Enhanced step instructions (1-step.md, 2-step.md, 3-step.md) with clearer guidance - Streamlined review content in x-review.md - Updated VS Code settings configuration - Added calculator operation screenshots for documentation
- Added /session and /share gist command examples - Included tip explaining the purpose of these commands - Renumbered steps for proper flow
There was a problem hiding this comment.
Pull request overview
This pull request updates the Copilot CLI exercise to improve clarity and usability for users learning to use the Copilot CLI in a Codespace environment. The changes focus on enhancing instructional steps, adding detailed Copilot CLI command usage with image-based examples, streamlining the Codespace environment for terminal-focused workflow, and expanding the exercise from 3 to 4 steps with comprehensive pull request workflows.
Changes:
- Enhanced exercise steps with image-based examples and clearer prompts for Copilot CLI interactions
- Added a new Step 4 for creating, reviewing, and merging pull requests
- Removed starter files (
package.json,calculator.js) to focus on CLI-generated code - Configured VS Code settings to create a terminal-focused Codespace environment
- Updated workflow version references from v0.7.0 to v0.7.3
Reviewed changes
Copilot reviewed 15 out of 18 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
.github/steps/1-step.md |
Enhanced step 1 with authentication instructions, /login and /session commands, and improved prompt formatting |
.github/steps/2-step.md |
Added branch creation activity, image-based calculator generation, and comprehensive testing instructions |
.github/steps/3-step.md |
Updated with image-based test expansion, improved prompt formatting, and clearer git workflow |
.github/steps/4-step.md |
New step for PR creation, code review with Copilot, issue linking, and merge workflow |
.github/steps/x-review.md |
Updated recap with 5-step structure and refreshed reference links |
.github/workflows/*.yml |
Updated workflow versions, added Step 3 workflow, modified Step 4 trigger from issues to PR closure |
.vscode/settings.json |
New file configuring terminal-focused Codespace environment with disabled UI elements |
calculator.js |
Removed starter file to let users generate code with Copilot CLI |
package.json |
Removed starter file to let users generate code with Copilot CLI |
images/*.png |
Added binary image files for visual examples in instructions |
.github/instructions/copilot-cli-ref.instructions.md |
New reference document for Copilot CLI commands and shortcuts |
Comments suppressed due to low confidence (2)
.vscode/settings.json:30
- The setting 'workbench.startupEditor' is duplicated on lines 8 and 30. Remove the duplicate entry.
"workbench.startupEditor": "none",
.vscode/settings.json:37
- The setting 'workbench.panel.opensMaximized' is duplicated on lines 29 and 37. Remove the duplicate entry.
"workbench.panel.opensMaximized": "always",
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| >- Expand tests based on the following example: | ||
| > - @images/calc-basic-operations.png | ||
| > - Add these tests to a src/tests/calculator.test.js file | ||
| > - Use a popular Node.js testing framework if one isn't installed | ||
| > - addition, subtraction, multiplication, and division |
There was a problem hiding this comment.
Inconsistent list formatting: line 126 uses >- while line 127 uses > -. This should be consistent for better markdown readability.
| >- Expand tests based on the following example: | |
| > - @images/calc-basic-operations.png | |
| > - Add these tests to a src/tests/calculator.test.js file | |
| > - Use a popular Node.js testing framework if one isn't installed | |
| > - addition, subtraction, multiplication, and division | |
| > - Expand tests based on the following example: | |
| > - @images/calc-basic-operations.png | |
| > - Add these tests to a src/tests/calculator.test.js file | |
| > - Use a popular Node.js testing framework if one isn't installed | |
| > - addition, subtraction, and multiplication, and division |
| > Add tests for the new calculator operations: | ||
| > - Expand tests based on the following example: |
There was a problem hiding this comment.
Trailing whitespace on line 107 after the colon. Should be removed for consistency.
|
|
||
| - **Custom agents**: Create specialized agent personas in `.github/agents/` for domain-specific workflows | ||
| - **`/delegate` command**: Assign tasks to Copilot coding agent for autonomous work | ||
| - **MCP server support**: Integrate with Model Context Protocol tools for extended functionality |
There was a problem hiding this comment.
Line 32 is duplicated from line 25. This creates redundancy in the feature list.
| - **MCP server support**: Integrate with Model Context Protocol tools for extended functionality |
| "workbench.editor.showTabs": "none", | ||
| "breadcrumbs.enabled": false, | ||
| "editor.minimap.enabled": false, | ||
| "workbench.startupEditor": "none", |
There was a problem hiding this comment.
The setting 'workbench.startupEditor' is duplicated on lines 8 and 30. Remove the duplicate entry.
This issue also appears in the following locations of the same file:
- line 30
| "workbench.editor.closeOnFileDelete": true, | ||
| "workbench.layoutControl.enabled": false, | ||
| "workbench.layoutControl.type": "hidden", | ||
| "workbench.panel.opensMaximized": "always", |
There was a problem hiding this comment.
The setting 'workbench.panel.opensMaximized' is duplicated on lines 29 and 37. Remove the duplicate entry.
This issue also appears in the following locations of the same file:
- line 37
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This pull request updates the Copilot CLI exercise to improve clarity, guidance, and usability for users learning to use the Copilot CLI in a Codespace environment. The changes focus on enhancing the instructional steps, adding detailed Copilot CLI command usage, and streamlining the Codespace environment for a more focused terminal-based workflow.