Skip to content

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

Merged

Conversation

tompratt-AQ
Copy link
Contributor

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

@dnfclas
Copy link

dnfclas commented Mar 24, 2017

@tompratt-AQ,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Copy link
Contributor

@mairaw mairaw left a 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
Copy link
Contributor

@mairaw mairaw Mar 24, 2017

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();
```
Copy link
Contributor

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.

Copy link
Contributor Author

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.

@mairaw mairaw merged commit 400dfda into dotnet:master Mar 24, 2017
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.

3 participants