-
Notifications
You must be signed in to change notification settings - Fork 844
[main] Source code updates from dotnet/dotnet #19216
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
base: main
Are you sure you want to change the base?
Changes from all commits
f9d8457
3feee3d
441b53e
72e56c6
c409217
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -117,10 +117,11 @@ type LanguageVersion(versionText) = | |
| static let languageVersion80 = 8.0m | ||
| static let languageVersion90 = 9.0m | ||
| static let languageVersion100 = 10.0m | ||
| static let languageVersion110 = 11.0m | ||
| static let previewVersion = 9999m // Language version when preview specified | ||
| static let defaultVersion = languageVersion100 // Language version when default specified | ||
| static let defaultVersion = languageVersion110 // Language version when default specified | ||
|
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. Shouldn't this be updated closer to the release time? We probably don't want to change the default so early. If I understand it correctly, FCS clients will get the effectively preview language version as default in all projects, unless some language version is specified.
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. I have put this on hold for main for now - I want to fly this config into NET11 SDK preview, so that preview users of NET11SDK do get langversion 11. But not in other channels, those should stay on 10 as the default.
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. My comment was about the tooling using FCS, not about the SDKs. I think we've never changed the default so early previously, and this would be quite unexpected. |
||
| static let latestVersion = defaultVersion // Language version when latest specified | ||
| static let latestMajorVersion = languageVersion100 // Language version when latestmajor specified | ||
| static let latestMajorVersion = languageVersion110 // Language version when latestmajor specified | ||
|
|
||
| static let validOptions = [| "preview"; "default"; "latest"; "latestmajor" |] | ||
|
|
||
|
|
@@ -135,6 +136,7 @@ type LanguageVersion(versionText) = | |
| languageVersion80 | ||
| languageVersion90 | ||
| languageVersion100 | ||
| languageVersion110 | ||
| |] | ||
|
|
||
| static let features = | ||
|
|
@@ -241,6 +243,12 @@ type LanguageVersion(versionText) = | |
| LanguageFeature.ReturnFromFinal, languageVersion100 | ||
| LanguageFeature.ErrorOnInvalidDeclsInTypeDefinitions, languageVersion100 | ||
|
|
||
| // F# 11.0 | ||
| // Put stabilized features here for F# 11.0 previews via .NET SDK preview channels | ||
|
|
||
| // Difference between languageVersion110 and preview - 11.0 gets turned on automatically by picking a preview .NET 11 SDK | ||
| // previewVersion is only when "preview" is specified explicitly in project files and users also need a preview SDK | ||
|
|
||
| // F# preview (still preview in 10.0) | ||
| LanguageFeature.FromEndSlicing, previewVersion // Unfinished features --- needs work | ||
| ] | ||
|
|
@@ -268,6 +276,8 @@ type LanguageVersion(versionText) = | |
| | "9" -> languageVersion90 | ||
| | "10.0" | ||
| | "10" -> languageVersion100 | ||
| | "11.0" | ||
| | "11" -> languageVersion110 | ||
| | _ -> 0m | ||
|
|
||
| let specified = getVersionFromString versionText | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,4 +10,5 @@ latestmajor | |
| 7.0 | ||
| 8.0 | ||
| 9.0 | ||
| 10.0 (Default) | ||
| 10.0 | ||
| 11.0 (Default) | ||
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.
DO NOT MERGE YET, VS MAPPING TO DECIDE