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

extra brace #477

Closed
wants to merge 1 commit into
base: live
from

Conversation

Projects
None yet
2 participants
@toshi0607
Contributor

toshi0607 commented Mar 28, 2018

No description provided.

public class CustomObject {
public String name {get; set;}
}
}

This comment has been minimized.

@toshi0607

toshi0607 Mar 28, 2018

Contributor

This is the only line I deleted.

@v-maudel

This comment has been minimized.

Contributor

v-maudel commented Mar 28, 2018

Hello, @toshi0607 ,

Thank for your contribution. We are currently reviewing it. We will communicate the decision taken accordingly.

Thanks for your time and support on this.

Kind regards,

Microsoft C&E Localization Team

@v-maudel v-maudel added Accepted and removed In Progress labels Apr 23, 2018

@v-maudel

This comment has been minimized.

Contributor

v-maudel commented Apr 23, 2018

Hello, @toshi0607 ,

Thank you for your feedback and suggestions.

This proposed file change has been accepted. The suggestion will be merged to this repository. We will release this article on the next localization release round.

Many thanks for your support.

Kind regards,

Microsoft C&E Localization Team

@v-maudel v-maudel closed this Apr 23, 2018

@toshi0607

This comment has been minimized.

Contributor

toshi0607 commented Jun 12, 2018

@v-maudel It seems this PR is just closed(not merged). Are there any problems with the way to send PR?
White Spaces problem?

@v-maudel

This comment has been minimized.

Contributor

v-maudel commented Jun 12, 2018

Hi @toshi0607 ,

The way you submitted the PR is OK and it was accepted.
Although , when the contribution is only about source code, we don't merge it because it needs to be modified in the source file first.
In any case, this was already updated and your feedback will be applied when we localize the new version. We will publish the new version of this article very soon.

You can check the PR we created to submit your change here:
MicrosoftDocs/azure-docs-pr#37694

Regards.
Microsoft C&E Localization Team

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment