Skip to content
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

Support .NET 7 SDKs #2377

Merged
merged 14 commits into from
Apr 12, 2022
Merged

Support .NET 7 SDKs #2377

merged 14 commits into from
Apr 12, 2022

Conversation

JoeRobich
Copy link
Member

@JoeRobich JoeRobich commented Apr 4, 2022

Adds new SdkOptions allowing users to specify a SDK path, a SDK version, and the ability to include prerelease SDKs during MSBuild locating.

In order to discover .NET 7.0 preview SDKs our hosts need to run on .NET 7.0 runtime

  • Tests now build against net7.0 so that the test runner will run them on the 7.0 runtime.
  • O# hosts still target net6.0 but now rollforward in a way that includes .NET 7.0 preview runtimes.

@JoeRobich JoeRobich marked this pull request as ready for review April 11, 2022 05:49
@JoeRobich JoeRobich requested a review from filipw April 11, 2022 05:49
@@ -161,7 +161,7 @@ public async Task Returns_struct_kind(string filename)

var usages = await FindSymbolsAsync(code, filename);
var symbols = usages.QuickFixes.Cast<SymbolLocation>().Select(q => q.Kind);
Assert.Equal("Structure", symbols.First());
Assert.Contains(symbols.First(), new[] { "Struct", "Structure" });
Copy link
Member Author

Choose a reason for hiding this comment

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

In the TypeKind enum both Struct and Structure have the same value. A runtime change must have effected the order they are returned. Either is valid since they both represent the value 10.

Copy link
Member

@filipw filipw left a comment

Choose a reason for hiding this comment

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

thanks!

@filipw filipw merged commit 04a561c into OmniSharp:master Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants