Skip to content
This repository was archived by the owner on Apr 20, 2023. It is now read-only.

Conversation

@dotnet-bot
Copy link

This is an automatically generated pull request from release/2.1.3xx into master.

git fetch --all
git checkout merges/release/2.1.3xx-to-master
git reset --hard upstream/master
git merge upstream/release/2.1.3xx
# Fix merge conflicts
git commit
git push merges/release/2.1.3xx-to-master --force

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

Livar and others added 2 commits March 16, 2018 17:32
Merge master into release/2.1.3xx
Other than change source to source-feed and make it additional instead of exclusive. I changed source to be multiple. Because restore support multiple source microsoft/dotnet#361

As for mock. The offline feed and source feed is considered the same, so remove the category of “source”. I renamed source to “AdditionalFeed” because that is more accurate on implementation level.

Note:
NuGet feed don’t have order. Whichever responses the fastest, is the first.
No change on restore.

scripts/cli-test-env.sh change is due to mac 10.13 is finally added to RID graph. And it is “considered” one of the CLI supported RID
@johnbeisner
Copy link

@wli3
Please review the InstallTool changes.

@wli3
Copy link

wli3 commented Mar 20, 2018

Should i review of fix? There is still merge conflicts

@wli3
Copy link

wli3 commented Mar 20, 2018

let me just fix it

@wli3
Copy link

wli3 commented Mar 20, 2018

@johnbeisner I merged master to merge branch again, the conflict automatically resolved. The diff for tools looks correct cc @peterhuene

@wli3
Copy link

wli3 commented Mar 21, 2018

@dotnet/dotnet-cli I merged master back to the merge branch. I think it is right thing. Help me double check

@wli3
Copy link

wli3 commented Mar 22, 2018

@dotnet/dotnet-cli ping

@johnbeisner johnbeisner merged commit f44b0e2 into master Mar 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants