Skip to content
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

Merge master to features/readonly-members #33485

Merged

Conversation

dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from master into features/readonly-members.

git fetch --all
git checkout merges/master-to-features/readonly-members
git reset --hard upstream/features/readonly-members
git merge upstream/master
# Fix merge conflicts
git commit
git push upstream merges/master-to-features/readonly-members --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

jaredpar and others added 4 commits February 18, 2019 10:27
The name for the named pipe used between MSBuild and VBCSCompiler is
generated from a combination of values including the current user name,
specifically `%USERNAME%`. It is possible for this value to have spaces
in it and hence the argument must be quoted when passing it to the
command line of VBCSCompiler instances.

Regression initially introduced: #32257
Ensure pipename argument quoted
… than HashSize (#33363)

* replace SHA1 to SHA256

* tweak

* Update src/Workspaces/Core/Portable/Workspace/Solution/Checksum.cs

Co-Authored-By: heejaechang <heejaechang@outlook.com>

* Update src/Workspaces/Core/Portable/Workspace/Solution/Checksum.cs

Co-Authored-By: heejaechang <heejaechang@outlook.com>

* Update src/Workspaces/Core/Portable/Workspace/Solution/Checksum.cs

Co-Authored-By: heejaechang <heejaechang@outlook.com>

* tweaks based on PR feedback

* removed unnecessary unsafe

* updated comment

* moved comments around and removed reference to SHA in checksum
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@ghost ghost merged commit 1a26c3e into features/readonly-members Feb 19, 2019
@ghost ghost deleted the merges/master-to-features/readonly-members branch February 19, 2019 13:30
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants