-
Notifications
You must be signed in to change notification settings - Fork 6k
Update language identifiers and styleguide. #1796
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
Update language identifiers and styleguide. #1796
Conversation
@tompratt-AQ, |
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.
Looks great @tompratt-AQ. Left a few comments.
@@ -25,7 +25,7 @@ The key concept to understand is that an async expression’s type is `Async<'T> | |||
|
|||
For example, say you wanted to download the HTML from dotnetfoundation.org without blocking the main thread. You can accomplish it like this: | |||
|
|||
```fs | |||
```fsharpharp |
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.
should be fsharp only. maybe search your working branch to see if you have other instances of fsharpharp or csharpharp
@@ -40,7 +40,7 @@ translation.priority.ht: | |||
# Static Classes and Static Class Members (C# Programming Guide) | |||
A [static](../../../csharp/language-reference/keywords/static.md) class is basically the same as a non-static class, but there is one difference: a static class cannot be instantiated. In other words, you cannot use the [new](../../../csharp/language-reference/keywords/new.md) keyword to create a variable of the class type. Because there is no instance variable, you access the members of a static class by using the class name itself. For example, if you have a static class that is named `UtilityClass` that has a public method named `MethodA`, you call the method as shown in the following example: | |||
|
|||
```cs | |||
```csharp | |||
UtilityClass.MethodA(); | |||
``` |
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.
the snippet below this one is missing the identifier. please add.
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 fixed these two issues and did not find any other instances like these.
Title
Update language identifiers and style guide for supported languages.
Summary
Added a list of supported languages with language IDs and markdown labels to the style guide. Fixed language identifiers for C#, F# and PHP in all files to match the supported markdown labels.
Fixes #1737
Suggested Reviewers
@mairaw