Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive documentation for C# and C++ tech stacks, introducing beginner-friendly reference guides for both languages that cover essential tools, frameworks, and best practices for modern development.
Key Changes:
- Added C# tech stack documentation covering .NET SDK, ASP.NET Core, and related tooling
- Added C++ tech stack documentation covering compilers, CMake, and modern C++ practices
- Included practical code examples, CI/CD configurations, and best practices for both languages
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docs/c-sharp/c-sharp-basics.md | New documentation file providing a comprehensive overview of the C#/.NET tech stack with examples and best practices |
| docs/c++/cpp-basics.md | New documentation file providing a comprehensive overview of the C++ tech stack with examples and best practices |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| @@ -0,0 +1,126 @@ | |||
| // ...existing code... | |||
There was a problem hiding this comment.
Remove the placeholder comment '// ...existing code...' on line 1. This is a comment artifact that doesn't belong in a new file and adds no value to the documentation.
| // ...existing code... |
| using Xunit; | ||
|
|
||
| public class Calculator { | ||
| public int Add(int a,int b) => a + b; |
There was a problem hiding this comment.
Add a space after the comma in the parameter list. The method signature should be 'Add(int a, int b)' to follow C# formatting conventions and improve readability.
| public int Add(int a,int b) => a + b; | |
| public int Add(int a, int b) => a + b; |
| int add(int a, int b) { return a + b; } | ||
|
|
||
| TEST_CASE("addition works") { | ||
| REQUIRE(add(2,3) == 5); |
There was a problem hiding this comment.
Add a space after the comma in the function call. It should be 'add(2, 3)' to follow C++ formatting conventions and improve readability.
| REQUIRE(add(2,3) == 5); | |
| REQUIRE(add(2, 3) == 5); |
📋 Pull Request Description
What does this PR do?
Type of Change
Related Issue
Closes #(issue number)
Changes Made
Documentation Location (if applicable)
docs/folder/file.mddocs/c-sharp/c-sharp-basics.md✅ Checklist
General
Documentation (if applicable)
Code Changes (if applicable)
Commit Messages
📸 Screenshots (if applicable)
🧪 Testing
How to Test
# Example commands to test npm link devdocx javascript/arraysTest Results
📚 Additional Context
🎃 Hacktoberfest
Note to Reviewers:
📝 Before Merging