Skip to content

Add a new csharp file#9368

Merged
BillWagner merged 3 commits intodotnet:masterfrom
kendrahavens:patch-1
Dec 5, 2018
Merged

Add a new csharp file#9368
BillWagner merged 3 commits intodotnet:masterfrom
kendrahavens:patch-1

Conversation

@kendrahavens
Copy link
Copy Markdown
Contributor

Summary

Add a section on how to add a new .cs file to a project. If approved, we should add the same to the VB doc.

Fixes #9155

Reviewers

@KathleenDollard

@kendrahavens kendrahavens requested a review from mairaw as a code owner December 4, 2018 03:17
Copy link
Copy Markdown
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This is a great addition @kendrahavens

I had one suggestion, if you wanted to update the code before you :shipit:. Overall, LGTM.

Comment thread docs/core/tutorials/with-visual-studio-code.md Outdated
Comment thread docs/core/tutorials/with-visual-studio-code.md Outdated
@kendrahavens
Copy link
Copy Markdown
Contributor Author

@BillWagner or @mairaw I don't have permissions to merge. Thank you for reviewing!

> [!TIP]
> For more information and troubleshooting tips on .NET Core debugging with OmniSharp in Visual Studio Code, see [Instructions for setting up the .NET Core debugger](https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger.md).

## Add a Class
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sentence case for headings

Suggested change
## Add a Class
## Add a class

I can merge for you and add VB later.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yay! Thank you!

@@ -82,6 +82,50 @@ You can also watch a short video tutorial for further setup help on [Windows](ht
> [!TIP]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also ms.date on line 6 should be updated.

@BillWagner BillWagner merged commit 7192124 into dotnet:master Dec 5, 2018
@kendrahavens
Copy link
Copy Markdown
Contributor Author

Thank you all!!

@kendrahavens kendrahavens deleted the patch-1 branch December 5, 2018 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants