-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Add description of debugging test unit #20494
Conversation
When should I update ms.date? |
I do not believe that we should make these changes. The example output is just that, and with these changes horizontal scrolling is introduced. We try to avoid that where possible. Thank you for taking the time to propose these changes, but I'd prefer to keep it as is. I'm going to close this for now, let me know if you want to discuss it further. |
@IEvangelist Thank you for check my request. |
Hi @johypark, The Also, I apologize about the misunderstanding. When I initially looked at this, it appeared as though you were removing the new line and making it one larger line which would have introduced scrolling. |
@IEvangelist Thank you for your answer which I clearly understand. |
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.
Minor nits.
Co-authored-by: David Pine <david.pine@microsoft.com>
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 good to me, we'll - thank you 🙏
Summary
Fix minor example of output
Add description of debugging test unit