Skip to content

Conversation

@mairaw
Copy link
Contributor

@mairaw mairaw commented Jun 29, 2018

No description provided.

JRAlexander and others added 9 commits June 28, 2018 09:54
* Added section on character literals

* Addressed review comments

* Escaped character
Hello, @ncarandini ,
This proposed file change comes from https://github.com/dotnet/docs.zh-cn/pull/158 .
Could you review this contribution and help to merge if agreed?
Many thanks in advance.
* latest version

* Interim checkin

* Interim checkin

* Finished draft

* Addressed review comments

* Added section on case

* Additional changes

* Incorporated review comments

* Addressed review comments

* Fixed broken xrefs
In https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/nullable-types/, assignment statements in the overview section was inconsistent (didn't end with a semi-colon or ended with a period)

This change is to fix them so that every statement in the section is a valid statement.
* Add item to stop using APTCA 

And changed tense

* Let's keep Bill happy

* some other minor changes

* forgot one heading

* fix metadata
* update foreach for `ref` variables.

* respond to feedback.
* C26886: Misuse of ` character to create codeboxes

Hello, @mairaw ,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.  
"this bug is affected by a source issue by the ´ character used to create codeboxes"
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR  then share your PR number so we can confirm and close this PR.

Many thanks in advance.

* feedback
@mairaw mairaw requested a review from BillWagner as a code owner June 29, 2018 04:50
@mairaw mairaw removed the request for review from BillWagner June 29, 2018 04:50
@BillWagner BillWagner merged commit 979597c into live Jun 29, 2018
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.

10 participants