Skip to content

Conversation

@rabryst
Copy link
Contributor

@rabryst rabryst commented May 24, 2018

On macOS and other operating systems with case-sensitive file systems, the docker command will not find the WeatherMicroService.cs file.

Summary

Changed WeatherMicroservice to WeatherMicroService in the Dockerfile.

On macOS and other operating systems with case-sensitive file systems, the docker command will not find the WeatherMicroService.cs file. Also corrected the output DLL.
@rabryst rabryst requested a review from BillWagner as a code owner May 24, 2018 19:37
@JRAlexander JRAlexander added the ✨ 1st-time docs contributor! Indicates PRs from new contributors to the docs repository label May 24, 2018
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.

Thanks for spotting this and submitting a fix @rabryst

We appreciate it.

I've reviewed the changes, and I'll :shipit: now.

You should see the changes on the live site in the next day or two on our regular publish cycle. Thanks again.

@BillWagner BillWagner merged commit 6ced683 into dotnet:master May 25, 2018
@rabryst rabryst deleted the patch-1 branch July 25, 2018 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 1st-time docs contributor! Indicates PRs from new contributors to the docs repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants