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

Update Compiler Server.md #31435

Merged
merged 2 commits into from Dec 4, 2018

Conversation

Projects
None yet
4 participants
@zuckerthoben
Contributor

zuckerthoben commented Nov 29, 2018

[ADDED] Description on how to set the environment variable on Windows. This is directly related to my PR #31433 because currently only file paths are supported on windows, although directory paths should also be supported.

Update Compiler Server.md
[ADDED] Description on how to set the environment variable on Windows. This is directly related to my PR #31433 because currently only file paths are supported on windows, although directory paths should also be supported.
@jcouv

jcouv approved these changes Nov 30, 2018

LGTM Thanks
(nit: I removed a doubled up empty line)

@jcouv jcouv added this to the 16.0.P2 milestone Nov 30, 2018

@jcouv jcouv self-assigned this Nov 30, 2018

@jcouv jcouv added the Documentation label Nov 30, 2018

@jaredpar jaredpar merged commit 04e4811 into dotnet:master Dec 4, 2018

3 of 5 checks passed

roslyn-integration-CI #20181129.19 failed
Details
windows_release_vs-integration_prtest Build finished.
Details
license/cla All CLA requirements met.
roslyn-CI #20181129.42 succeeded
Details
windows_debug_vs-integration_prtest Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment