-
Notifications
You must be signed in to change notification settings - Fork 524
Allow to restore netcoreapp5.0 #3812
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
| | DotNetCoreApp DotNetCoreAppVersion.V2_2 -> [ DotNetCoreApp DotNetCoreAppVersion.V2_1 ] | ||
| | DotNetCoreApp DotNetCoreAppVersion.V3_0 -> [ DotNetCoreApp DotNetCoreAppVersion.V2_2; DotNetStandard DotNetStandardVersion.V2_1 ] | ||
| | DotNetCoreApp DotNetCoreAppVersion.V3_1 -> [ DotNetCoreApp DotNetCoreAppVersion.V3_0 ] | ||
| | DotNetCoreApp DotNetCoreAppVersion.V5_0 -> [ DotNetCoreApp DotNetCoreAppVersion.V3_1; DotNetFramework FrameworkVersion.V4_8 ] |
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.
Are you sure that DotNetFramework FrameworkVersion.V4_8 should be here? (Just asking)
|
Isn't that what you sent me in the youtube link?
Sergey Tihon <[email protected]> schrieb am Mi., 18. März 2020,
08:56:
… ***@***.**** commented on this pull request.
------------------------------
In src/Paket.Core/Versioning/FrameworkHandling.fs
<#3812 (comment)>:
> @@ -650,6 +656,7 @@ type FrameworkIdentifier =
| DotNetCoreApp DotNetCoreAppVersion.V2_2 -> [ DotNetCoreApp DotNetCoreAppVersion.V2_1 ]
| DotNetCoreApp DotNetCoreAppVersion.V3_0 -> [ DotNetCoreApp DotNetCoreAppVersion.V2_2; DotNetStandard DotNetStandardVersion.V2_1 ]
| DotNetCoreApp DotNetCoreAppVersion.V3_1 -> [ DotNetCoreApp DotNetCoreAppVersion.V3_0 ]
+ | DotNetCoreApp DotNetCoreAppVersion.V5_0 -> [ DotNetCoreApp DotNetCoreAppVersion.V3_1; DotNetFramework FrameworkVersion.V4_8 ]
Are you sure that DotNetFramework FrameworkVersion.V4_8 should be here?
(Just asking)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3812 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAOANEZHG64GN6VP5Y4EWDRIB5EJANCNFSM4LOJBKFA>
.
|
|
I read this line as "There will be backward compatibility between DotNetCoreAppVersion.V5_0 and FrameworkVersion.V4_8" and that is probably not true |
|
Yes the line reads as ".net 4.8 libs can be used in .net 5 apps". I thought
that's the whole idea. /cc @matthid
Sergey Tihon <[email protected]> schrieb am Mi., 18. März 2020,
09:38:
… I read this line as "There will be backward compatibility between
DotNetCoreAppVersion.V5_0 and FrameworkVersion.V4_8" and that is probably
not true
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3812 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAOANB6XWJJLUR3DK3ENELRICB7HANCNFSM4LOJBKFA>
.
|
|
From the design document:
Also, under design, it mentions that
|
|
Basically, |
|
thx. |
fixes #3811
/cc @kerams