Skip to content

Conversation

@svick
Copy link
Contributor

@svick svick commented Jun 8, 2018

Fixes #5700.
Fixes dotnet/core#1669.
Fixes #2648.

Companion PR that updates samples: dotnet/samples#114.

@pkulikov
Copy link
Contributor

pkulikov commented Jun 8, 2018

@svick does it also fix #2648?

@svick
Copy link
Contributor Author

svick commented Jun 8, 2018

@pkulikov Thanks, I didn't know about that issue. Yeah, I think this PR should cover that issue too, I have added it to the "fixes" list.

@rpetrusha rpetrusha added the 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) label Jun 8, 2018
@rpetrusha
Copy link
Contributor

Closing and reopening to begin new build.

@rpetrusha rpetrusha closed this Jun 9, 2018
@rpetrusha rpetrusha reopened this Jun 9, 2018
@mairaw
Copy link
Contributor

mairaw commented Jun 9, 2018

I see that the companion PR was merged and I was planning to push the samples live now, but I won't do that until this is merged. When we merge companion PRs, we should be ready to merge both as soon as builds pass.

@svick
Copy link
Contributor Author

svick commented Jun 9, 2018

@mairaw If it helps, feel free to revert that PR and I can recreate it.

@mairaw
Copy link
Contributor

mairaw commented Jun 9, 2018

Don't worry @svick. There weren't any high-pri changes to push so we can wait until next week.

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This is a really good set of changes @svick
I'll :shipit: now.

Thanks for all your contributions to docs.

@BillWagner BillWagner merged commit 66b7622 into dotnet:master Jun 11, 2018
@svick svick deleted the microservices-update branch June 11, 2018 16:58
@mairaw mairaw removed the 🚧 Hold for related PR Indicates a PR can only be merged when other related PRs are merged (see comments for links) label Nov 10, 2019
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.

NET Core Version Issues Docker setup and file Update C# Docker Tutorial

5 participants