-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Object.xml -- updating for Try .NET #3346
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
Conversation
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.
Thanks @WilliamAntonRohm. Most of my feedback is regarding the presence of the interactive note on specific pages and one change that needs to be undone.
@mairaw -- Maira, I believe I've addressed all your review comments. Thank you for your continued assistance. |
#sign-off |
Resolved some merge conflicts |
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.
I need to verify the build looks good and as expected but I think this one is ready to merge after the changes I made.
OPS build finished successfully so merging this. |
associated samples PR is dotnet/samples#1640