Skip to content

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Apr 11, 2019

I'll be submitted dotnet/samples#810 in the samples repo to match

@nschonni nschonni requested a review from Lxiamail as a code owner April 11, 2019 04:00
@nschonni nschonni changed the title fix: Underscore "astoria reflection provider" fix: Underscore astoria snippet paths Apr 11, 2019
nschonni added a commit to nschonni/dotnet-samples that referenced this pull request Apr 11, 2019
Related to dotnet/docs#11784
Some files were deleted because they are gitignored
rpetrusha pushed a commit to dotnet/samples that referenced this pull request Apr 11, 2019
Related to dotnet/docs#11784
Some files were deleted because they are gitignored
@rpetrusha
Copy link
Contributor

Closing and reopening to begin new build.

@rpetrusha rpetrusha closed this Apr 11, 2019
@rpetrusha rpetrusha reopened this Apr 11, 2019
@rpetrusha
Copy link
Contributor

@nschonni, the build report shows 19 warnings because of unresolved code references.

@nschonni
Copy link
Contributor Author

Looks like I missed some under "xml", I only renamed them under visualbasic and csharp. Not sure about the few csharp ones that are still erroring

@mairaw mairaw closed this Apr 11, 2019
@mairaw mairaw reopened this Apr 11, 2019
@mairaw
Copy link
Contributor

mairaw commented Apr 11, 2019

Starting a new build...

@mairaw
Copy link
Contributor

mairaw commented Apr 12, 2019

6 more warnings...

@nschonni
Copy link
Contributor Author

@mairaw thanks for fixing those up, I was just checking again to see if it was the samples again, but it looks like I messed up a a find/replace somewhere

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.

It's good now. Thanks @nschonni!

@mairaw mairaw merged commit 0de4832 into dotnet:master Apr 12, 2019
@mairaw mairaw added this to the April 2019 milestone Apr 12, 2019
@nschonni nschonni deleted the fix-astoria-snippets branch April 12, 2019 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants