CSharp Module Final Review #191
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…kflow explanations
…g detailed instructions on code completion and slash commands.
…Agent Mode, including coding tasks, context addition, and project improvements.
… Mode, including new API endpoint generation instructions and customization context.
… for Customization and Context
- Introduced a new settings.json file for cSpell configuration. - Revised section titles and content in README.md to enhance clarity. - Added images for better visual guidance in the README.
… clarity on running and debugging the backend project
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR contains the final review of a C# module for GitHub Copilot, restructuring the content into 4 comprehensive sections with updated screenshots and GIFs. The module teaches developers how to effectively use GitHub Copilot with C# through hands-on exercises covering code completion, agent mode, and customization features.
Key changes include:
- Complete reorganization from step-based to section-based learning structure
- Addition of new features like Agent Mode and custom instructions
- Updated visual assets (screenshots and GIFs) to reflect current UI
Reviewed Changes
Copilot reviewed 3 out of 31 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| Using-GitHub-Copilot-with-CSharp/README.md | Complete rewrite of the module content with 4 new sections, updated exercises, and modern GitHub Copilot features |
| Using-GitHub-Copilot-with-CSharp/011SlashCommandExplain.gif | Removed outdated GIF file |
| .vscode/settings.json | Added spell check configuration for "Blazor" |
Comments suppressed due to low confidence (1)
Using-GitHub-Copilot-with-CSharp/README.md:262
- The code comment appears to contain a spelling error: 'paremeter' should be 'parameter'.
By supplying Copilot with detailed project context, instructions files significantly improve the relevance and accuracy of its code suggestions. For example, if your project uses Blazor and ASP.NET Core, Copilot can generate components that follow your preferred structure, use modern C# features, and adhere to your naming conventions. This leads to more consistent code and reduces the need for manual corrections or lengthy code reviews.
| <img src="./images/003RunBackEndProject.jpg" alt="Run and Debug in the Panel"> | ||
| </div> | ||
|
|
||
| 2. The BackEnd project for our Weather API will now be running in port 8080. If you used Copilot to Run and Debug, it will output the URL for you in the chat pane. Simply click the text and it will open a new tab with the published URL. Ensure that `/weatherforcast` is in the published URL. |
There was a problem hiding this comment.
The word 'weatherforcast' is misspelled. It should be 'weatherforecast' to match the correct endpoint name used elsewhere in the documentation.
Suggested change
| 2. The BackEnd project for our Weather API will now be running in port 8080. If you used Copilot to Run and Debug, it will output the URL for you in the chat pane. Simply click the text and it will open a new tab with the published URL. Ensure that `/weatherforcast` is in the published URL. | |
| 2. The BackEnd project for our Weather API will now be running in port 8080. If you used Copilot to Run and Debug, it will output the URL for you in the chat pane. Simply click the text and it will open a new tab with the published URL. Ensure that `/weatherforecast` is in the published URL. |
| 2. The BackEnd project for our Weather API will now be running in port 8080. If you used Copilot to Run and Debug, it will output the URL for you in the chat pane. Simply click the text and it will open a new tab with the published URL. Ensure that `/weatherforcast` is in the published URL. | ||
|
|
||
| <div align="left"> | ||
| <img src="./images/004rundebugURL.jpg" alt="Run and Debug Output" |
There was a problem hiding this comment.
The image tag is missing the closing > and width/height attributes for consistency with other images in the document.
Suggested change
| <img src="./images/004rundebugURL.jpg" alt="Run and Debug Output" | |
| <img src="./images/004rundebugURL.jpg" alt="Run and Debug Output" width="500" height="300"> |
|
|
||
| Add a breakpoint in line 24 (press F9) and refresh the browser with the Url to test the endpoint. The browser should not show the weather forecast, and in the Visual Studio Editor we can see how the program execution was paused at line 24. | ||
| ```csharp | ||
| // create a new internal record named WeatherForecastByCity that request the following parameters: City, Date, TemperatureC, Summary |
There was a problem hiding this comment.
The word 'request' should be 'requests' to maintain proper grammar in the comment.
Suggested change
| // create a new internal record named WeatherForecastByCity that request the following parameters: City, Date, TemperatureC, Summary | |
| // create a new internal record named WeatherForecastByCity that requests the following parameters: City, Date, TemperatureC, Summary |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
C Sharp Module ready for final review: