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

Add .NET 6 and C# 10 Support #334

Merged
merged 8 commits into from
Nov 9, 2021
Merged

Add .NET 6 and C# 10 Support #334

merged 8 commits into from
Nov 9, 2021

Conversation

RehanSaeed
Copy link
Owner

@RehanSaeed RehanSaeed commented Oct 15, 2021

Will update the .editorconfig in a later PR and go with file scoped namespaces.

@RehanSaeed RehanSaeed added enhancement Issues describing an enhancement or pull requests adding an enhancement. major Pull requests requiring a major version update according to semantic versioning. labels Oct 15, 2021
@RehanSaeed
Copy link
Owner Author

@Turnerj I tried adding net6.0 to the tool project and found new warnings/errors but I could not then get the main Schem.NET project to build, so rolled that back but kept the fixes for the warning/errors. So the source generator can't target multiple target frameworks? Can we upgrade from netstandard2.0?

@Turnerj
Copy link
Collaborator

Turnerj commented Oct 15, 2021

Source generators always have to target .NET Standard 2.0 unfortunately - it is a limitation baked into how they are invoked I believe. Might change when/if VS is no longer using .NET Framework.

Benchmarks/Schema.NET.Benchmarks/SchemaBenchmarkBase.cs Outdated Show resolved Hide resolved
Tools/Schema.NET.Tool/Services/SchemaService.cs Outdated Show resolved Hide resolved
Tests/Schema.NET.Test/Schema.NET.Test.csproj Outdated Show resolved Hide resolved
Source/Schema.NET/Schema.NET.csproj Outdated Show resolved Hide resolved
Source/Common/ContextJsonConverter.cs Show resolved Hide resolved
@RehanSaeed RehanSaeed changed the title Add .NET 6 and C# 10 Add .NET 6 and C# 10 Support Oct 19, 2021
Copy link
Collaborator

@Turnerj Turnerj left a comment

Choose a reason for hiding this comment

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

LGTM!

@RehanSaeed RehanSaeed force-pushed the add-dotnet-6-and-csharp-10 branch from 5997177 to e4cea21 Compare November 9, 2021 09:48
@RehanSaeed RehanSaeed merged commit 61b629c into main Nov 9, 2021
@RehanSaeed RehanSaeed deleted the add-dotnet-6-and-csharp-10 branch November 9, 2021 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issues describing an enhancement or pull requests adding an enhancement. major Pull requests requiring a major version update according to semantic versioning.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants