Skip to content

Comments

Move IDE code over to C# 14.#79835

Merged
CyrusNajmabadi merged 7 commits intodotnet:mainfrom
CyrusNajmabadi:csharp14
Dec 9, 2025
Merged

Move IDE code over to C# 14.#79835
CyrusNajmabadi merged 7 commits intodotnet:mainfrom
CyrusNajmabadi:csharp14

Conversation

@CyrusNajmabadi
Copy link
Contributor

No description provided.

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner August 7, 2025 23:59
@CyrusNajmabadi CyrusNajmabadi requested review from jcouv and removed request for a team August 7, 2025 23:59
@CyrusNajmabadi
Copy link
Contributor Author

@dotnet/roslyn-ide ptal.


public static bool IsCSharp14OrAbove(this LanguageVersion languageVersion)
=> languageVersion >= (LanguageVersion)1400; // Update to LanguageVersion.CSharp14 when available.
=> languageVersion >= LanguageVersion.CSharp14;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can't make that change without updating a nuget package reference first.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup. This is just getting the PR ready for when that happens :)

Copy link
Member

@dibarbet dibarbet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems fine, but looks like there are some build failures

@CyrusNajmabadi
Copy link
Contributor Author

@davidwengier ptal.

@CyrusNajmabadi CyrusNajmabadi merged commit 4b0b168 into dotnet:main Dec 9, 2025
26 checks passed
@CyrusNajmabadi CyrusNajmabadi deleted the csharp14 branch December 9, 2025 20:05
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Dec 9, 2025
@davidwengier davidwengier modified the milestones: Next, 18.3 Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants