-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Updated index.md #1187
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
Updated index.md #1187
Conversation
ghost
commented
Jun 29, 2018
- Recast "for creating" and replaced with "to create" in Intro section.
- Added a bullet list for .NET Core projects in the .Net Core section.
- Fixed capitalization errors and removed extra spaces between words.
Recast "for creating" and replaced with "to create" in Intro section. Added a bullet list for .NET Core projects in the .Net Core section. Fixed capitalization errors and removed extra spaces between words.
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
@dsmatlak : Thanks for your contribution! The author, @asb3993, has been notified to review your proposed change. |
mac/index.md
Outdated
@@ -41,7 +41,12 @@ F# is a strongly-typed functional programming language designed to run on .NET. | |||
|
|||
## .NET Core | |||
|
|||
[.NET Core](https://www.microsoft.com/net/core#macos) is a platform for creating applications that run on Windows, Linux, and Mac. Visual Studio for Mac has support for loading, creating, running, and debugging .NET Core projects. | |||
[.NET Core](https://www.microsoft.com/net/core#macos) is a platform to create applications that run on Windows, Linux, and Mac. Visual Studio for Mac has support for these .NET Core projects: | |||
|
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.
I don't think putting this as a bulleted list makes sense. These are all actions that are supported, not types of projects.
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.
Thanks for the feedback. I will remove the bullet list.
Hey @dsmatlak, this looks great and I appreciate the additions. There's just one issue that I think needs to be fixed before this can be merged. |
Removed the bullet list from the .NET Core section.
@dsmatlak : Thanks for your contribution! The author, @asb3993, has been notified to review your proposed change. |
✅ Validation status: passed
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
Approved! This will be merged shortly. Thanks again. |