-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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 acc 8000 ount related emails.
Already on GitHub? Sign in to your account
Conversation
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.
@@ -2,7 +2,7 @@ | |||
|
|||
image: Visual Studio 2017 | |||
|
|||
# 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
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.