-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Move to the official .NET Core 2.0 #4603
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
Conversation
TravisEz13
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 Minor comment.
| # cache version - netcoreapp.2.0.0-preview3-25426-01 | ||
| # cache version - netcoreapp.2.0.0 | ||
| cache: | ||
| - '%LocalAppData%\Microsoft\dotnet -> appveyor.yml' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't we update the caches to depend on PowerShell.Common.props. Then we only have one place to update the version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some changes pending for PowerShell.Common.props (see Ilya's MSBuild refactoring PRs). I think for the time being, PowerShell.Common.props has more churns than appveyor.yml.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good
adityapatwardhan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@daxian-dbw This is a major change, should we run Feature tests? |
|
@adityapatwardhan I requested feature test run for the PR, please take a look at the second commit message. I will investigate failure in appveyor. |
|
There were 11 tests failing in AppVeyor and all are web cmdlet tests. They failed because of time out. |
.NET Core 2.0 was officially released on 8/14/2017.
This PR is to migrate our tooling and build to the official .NET Core 2.0 bits.