Restore netcoreapp1.0 support for testhost #2554
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sharwell
force-pushed
the
fix-netstandard1
branch
3 times, most recently
from
a055e62
to
b5ba8f0
Compare
sharwell
force-pushed
the
fix-netstandard1
branch
from
b5ba8f0
to
07d3c8e
Compare
|
All looks OK, but I am converting this to draft, I'll add new targets to NuGet packages and tests and then merge. |
Haplois
approved these changes
Sep 15, 2020
|
Hello @Haplois! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
ghost
merged commit 
|
@sharwell what's the reason for adding/restoring netcoreapp1.0 support? Wondering as netcoreapp1.0 has been EOL for a long time. |
|
@ViktorHofer It allows multi-targeting projects to move forward with adding support for the latest runtime targets without blocking adoption of new targets on the removal of an unrelated one. The issue I hit directly involved analyzer testing for Roslyn analyzers that support Visual Studio 2015, which is still supported. We could carve out slightly different cases over time, where the .NET Framework tools from 2015 keep working but the .NET Core tooling just breaks at some point, but that seemed unnecessary considering the ease with which .NET Core 1.x can be supported by the test runner. |
This was referenced Mar 6, 2021
This was referenced Mar 13, 2021
This was referenced Mar 16, 2021
mburumaxwell
pushed a commit
to faluapp/falu-dotnet
that referenced
this pull request
Jun 12, 2021
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 16.8.3 to 16.9.1. #Release notes *Sourced from [Microsoft.NET.Test.Sdk's releases](https://github.com/microsoft/vstest/releases).* > ## v16.9.1 > See the release notes [here](https://github.com/microsoft/vstest-docs/blob/master/docs/releases.md#1691). > > ## v16.9.0-preview-20210127-04 > See the release notes [here](https://github.com/microsoft/vstest-docs/blob/master/docs/releases.md#1690-preview-20210127-04). > > ## v16.9.0-preview-20210106-01 > See the release notes [here](https://github.com/microsoft/vstest-docs/blob/master/docs/releases.md#1690-preview-20210106-01). > > ## v16.9.0-preview-20201123-03 > See the release notes [here](https://github.com/microsoft/vstest-docs/blob/master/docs/releases.md#1690-preview-20201123-03). > > ------------------------------- > > ## v16.9.0-preview-20201020-06 > See the release notes [here](https://github.com/microsoft/vstest-docs/blob/master/docs/releases.md#1690-preview-20201020-06). > > ------------------------------- > > ## 16.9.0-preview-20201020-06 > > ### Issue Fixed > * Enable Fakes Datacollector settings to be added in design mode [#2586](microsoft/vstest#2586) > * Fix blame parameter, warning, and add all testhosts to be ngend [#2579](microsoft/vstest#2579) > * Add netcoreapp1.0 support to `Microsoft.TestPlatform.TestHost` NuGet [#2569](microsoft/vstest#2569) > * Use bitness from process or OS [#2571](microsoft/vstest#2571) > * Restore netcoreapp1.0 support for testhost [#2554](microsoft/vstest#2554) > * Get symbols of DiaSymReader from externals [#2560](microsoft/vstest#2560) > * Do not force .NET4.5 in case legacy test settings are provided [#2545](microsoft/vstest#2545) > * Simplify package references [#2559](microsoft/vstest#2559) > * Enable default compile items for Microsoft.TestPlatform.PlatformAbstractions [#2556](microsoft/vstest#2556) > * Avoid logging >Task returned false but did not log an error.< [#2557](microsoft/vstest#2557) > * Fixed code coverage compatibility issue [#2527](microsoft/vstest#2527) > * Add environment variables to enable MacOS dump > * Adding test run attachments processing [#2463](microsoft/vstest#2463) > > See full log [here](microsoft/vstest@v16.8.0-release-20200921-02...v16.9.0-preview-20201020-06) > > ### Drops > > * TestPlatform vsix: [16.9.0-preview-20201020-06](https://vsdrop.corp.microsoft.com/file/v1/Products/DevDiv/microsoft/vstest/master/20201020-06;/TestPlatform.vsix) > * Microsoft.TestPlatform.ObjectModel : [16.9.0-preview-20201020-06](https://www.nuget.org/packages/Microsoft.TestPlatform.ObjectModel/16.9.0-preview-20201020-06) ...
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request updates the build to include netcoreapp1.0 outputs for testhost.exe and testhost.x86.exe. It does not (yet) update tests to cover these cases, or NuGet packaging to include the output binaries.