-
Notifications
You must be signed in to change notification settings - Fork 6
Skullxcode patch 1 #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skullxcode patch 1 #13
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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... | |||
Copilot
AI
Oct 15, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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; |
Copilot
AI
Oct 15, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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); |
Copilot
AI
Oct 15, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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