Skip to content

Conversation

WilliamAntonRohm
Copy link
Contributor

@WilliamAntonRohm WilliamAntonRohm commented Jan 15, 2020

new samples PR is dotnet/samples#1916

replaces #3458

started with an up-to-date copy of String.xml & re-interactivated sample file references

@WilliamAntonRohm
Copy link
Contributor Author

@mairaw -- Maira, please review this (hopefully last of this series) version of String.xml, which includes all the new interactive sample refs.

@mairaw mairaw added the vendor-project Indicates the issue/pr is related to a vendor project. label Jan 16, 2020
Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @WilliamAntonRohm. I can't edit this file directly, so leaving review comments for you to address please.

Copy link
Contributor Author

@WilliamAntonRohm WilliamAntonRohm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(another unasked-for review from GitHub)

lgtm

@mairaw
Copy link
Contributor

mairaw commented Jan 17, 2020

New build now that samples change was merged.

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mairaw mairaw merged commit abc0627 into dotnet:master Jan 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vendor-project Indicates the issue/pr is related to a vendor project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants