-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Update index.md #1756
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 index.md #1756
Conversation
@Ishmeet-Singh-Saggu : Thanks for your contribution! The author, @, has been notified to review your proposed change. |
@Mikejo5000 Can you take a look at these suggestions? |
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.
@Ishmeet-Singh-Saggu The first edit is fine, but we need to stick to "Improve performance" for the second. "Optimize performance" is too formal based on current stylistic conventions.
@Ishmeet-Singh-Saggu "Measure performance" would be OK as an alternative. |
@Ishmeet-Singh-Saggu : Thanks for your contribution! The author, @, has been notified to review your proposed change. |
Thanks for suggestion. I have made the required changes as specified, please review it. |
Thanks for suggestion. I have made the required changes please review it.
Regards,
Ishmeet Singh Saggu
…On Tue, Oct 30, 2018 at 12:12 AM Mike Jones ***@***.***> wrote:
@Ishmeet-Singh-Saggu <https://github.com/Ishmeet-Singh-Saggu> "Measure
performance" would be OK as an alternative.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1756 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Amv7fCxnt5vUdq5Tj6ECWKCSHt0do2eHks5up0wcgaJpZM4X9ZGQ>
.
|
@Mikejo5000 Please review the changes and sign off if these are acceptable. Thank you! |
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.
LG. Thanks!
#sign-off |
@Mikejo5000: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias. |
@gewarren for sign-off |
A couple of minor changes