-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
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. |

Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.

.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.