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 for .NET 8 #28

Closed
paulirwin opened this issue Nov 20, 2023 · 5 comments · Fixed by #29
Closed

Support for .NET 8 #28

paulirwin opened this issue Nov 20, 2023 · 5 comments · Fixed by #29
Assignees

Comments

@paulirwin
Copy link
Contributor

.NET 8 LTS is now generally available, and the tool requires .NET 7 to be installed in a CI environment currently. Updating the project to support .NET 8 will help set it up for the next three years of LTS support.

@TAGC TAGC self-assigned this Nov 20, 2023
@TAGC
Copy link
Owner

TAGC commented Nov 20, 2023

Thanks for raising this Paul. I'd like to get your PR merged but it doesn't look like AppVeyor has a Visual Studio image that includes the .NET 8 SDK yet. I don't expect it will be too long before they release an image with .NET 8 included.

@paulirwin
Copy link
Contributor Author

@TAGC Thanks! What are your thoughts on removing .NET Core 2.1, .NET Core 3.1, and .NET 5 support now that they are out of support by Microsoft? Building locally gives lots of warnings about security vulnerabilities as a result of still targeting them.

@TAGC
Copy link
Owner

TAGC commented Nov 20, 2023

@paulirwin I think that's definitely a good shout. I'll probably release one version with just the .NET 8 upgrade in place and then another one that removes support for those three unsupported versions.

@TAGC
Copy link
Owner

TAGC commented Nov 21, 2023

Hi @paulirwin, I've released a prerelease package (3.0.0-alpha0004) that incorporates the changes from #31. I've tested it locally and it seems to work okay. I've changed my mind and think I might incorporate .NET 8 support (when it's ready) as part of the same upcoming stable release (3.0.0), rather than split them across two releases.

@paulirwin
Copy link
Contributor Author

@TAGC Sounds good, I just updated my .NET 8 PR with merging in the latest changes from develop.

@TAGC TAGC closed this as completed in #29 Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants