-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Removing "netcoreapp" and "netfx" TargetGroup #457
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
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
2eeecc8
netcoreapp -> netcoreapp5.0 in .sln files
Anipik 9bb38a8
netcoreapp; -> netcoreapp5.0; , netcoreapp- to netcoreapp5.0-
Anipik cc7489a
netcoreapp- -> netcoreapp5.0 in .csproj
Anipik b1e889c
'netcoreapp -> 'netcoreapp5.0 in .csproj
Anipik bd1891f
done with libraries sub folder
Anipik 2f70d3b
missed semicolon
Anipik 66bfc49
minor changes
Anipik fe33700
netcoreapp -> netcoreapp5.0 in some extra files
Anipik 7831a53
vNext target variable in configurations.props
Anipik 71b9862
netcoreapp => netcoreapp5/0 in some other .props file
Anipik bb47f23
netcoreapp -> vnext in .csproj
Anipik 9450fb5
making the netcorepap framework switch work
Anipik eb62bc5
fixing sendToHelix and linux build
Anipik ff73c41
Merge branch 'master' into vnext
Anipik e9839b0
TargetFrameworkVNext -> netcoreappCurrent
Anipik b25e16e
Merge branch 'master' into vnext
Anipik 04f85e3
case correct netcoreappcurrent
Anipik 731d5fb
fixing send to helix zip path
Anipik 41a8482
netfx -> net472 in .sln
Anipik b0fca9a
netfx; -> net472; in .props
Anipik 94db048
netfx- -> net472- in .props
Anipik b6a5789
netfx - -> net472- in .csproj
Anipik 10af485
'netfx-> 'net472 in .csproj
Anipik 0f7edf7
netfx -> net472 manual change
Anipik 73b3760
fixing ci build for netfx
Anipik 3f2787b
Not trying to change the global property
Anipik 66bd077
Merge branch 'master' into vnext
Anipik 80e4e24
Fixing build failure due to master changes in net https tests
Anipik 325f614
Merge branch 'master' into vnext
Anipik 1be30cf
adding documentaiton and fixing filtering
Anipik e5ee4eb
netcoreapp -> netcore5.0 in py file
Anipik 6de62e5
Merge branch 'master' into vnext
Anipik 9d4decb
changing the path to use netcoreapp5.0 version
Anipik 4e7f2ca
feedback and override corefx path corrected in installers
Anipik 5d53653
remaing netcoreapp5.0 changes done
Anipik b1ca85f
Merge branch 'master' into vnext
Anipik 596aa19
netcoreapp-> netcoreapp5.0 after live builds
Anipik f040d5c
net472 -> netframeworkcurrent
Anipik File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -120,8 +120,8 @@ jobs: | |
| - task: CopyFiles@2 | ||
| displayName: Prepare shared framework runtime folder to publish | ||
| inputs: | ||
| sourceFolder: $(Build.SourcesDirectory)/artifacts/bin/pkg/${{ parameters.framework }}/runtime | ||
| targetFolder: $(Build.ArtifactStagingDirectory)/artifacts/bin/pkg/${{ parameters.framework }}/runtime | ||
| sourceFolder: $(Build.SourcesDirectory)/artifacts/bin/pkg/netcoreapp5.0/runtime # The hardcoded target framework should be removed when we drop the support for versionless targetframeworks from ci. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FYI: @jkoritzinsky |
||
| targetFolder: $(Build.ArtifactStagingDirectory)/artifacts/bin/pkg/netcoreapp5.0/runtime | ||
|
|
||
| - task: CopyFiles@2 | ||
| displayName: Prepare docs folder to publish | ||
|
|
||
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| <Project> | ||
| <PropertyGroup> | ||
| <BuildConfigurations> | ||
| netcoreapp-Unix; | ||
| netcoreapp-Windows_NT; | ||
| $(NetCoreAppCurrent)-Unix; | ||
| $(NetCoreAppCurrent)-Windows_NT; | ||
| </BuildConfigurations> | ||
| </PropertyGroup> | ||
| </Project> |
4 changes: 2 additions & 2 deletions
4
src/libraries/Common/tests/CoreFx.Private.TestUtilities/Configurations.props
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| <Project> | ||
| <PropertyGroup> | ||
| <BuildConfigurations> | ||
| netcoreapp; | ||
| netfx; | ||
| $(NetCoreAppCurrent); | ||
| $(NetFrameworkCurrent); | ||
| </BuildConfigurations> | ||
| </PropertyGroup> | ||
| </Project> |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
2 changes: 1 addition & 1 deletion
2
src/libraries/Common/tests/System/Xml/XmlCoreTest/Configurations.props
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| <Project DefaultTargets="Build"> | ||
| <PropertyGroup> | ||
| <BuildConfigurations> | ||
| netcoreapp; | ||
| $(NetCoreAppCurrent); | ||
| </BuildConfigurations> | ||
| </PropertyGroup> | ||
| </Project> |
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.