Skip to content

Add .NET 6 and C# 10 Support#334

Merged
RehanSaeed merged 8 commits intomainfrom
add-dotnet-6-and-csharp-10
Nov 9, 2021
Merged

Add .NET 6 and C# 10 Support#334
RehanSaeed merged 8 commits intomainfrom
add-dotnet-6-and-csharp-10

Conversation

@RehanSaeed
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

@RehanSaeed RehanSaeed changed the title Add .NET 6 and C# 10 Add .NET 6 and C# 10 Support Oct 19, 2021
Copy link
Copy Markdown
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