poppastring / dasblog-core

The original DasBlog reimagined with ASP.NET Core
MIT License
469 stars 193 forks source link

Upgrading to dotnet7 #680

Closed tomlane closed 1 year ago

tomlane commented 1 year ago

Hello!

I have had a go at upgrading the solution for #648.

The projects have been updated to target net7.0, I've changed any references to directions from net6.0 to net 7.0 and upgraded nuget references that have an explicit upgrade path from 6.x to 7.x.

Following the upgrade, there was an error within automapper but that appears to be resolved by upgrading to the latest version.

I appreciate this is quite an ambitious first PR, so I am open to feedback and guidance. 😄

EDIT: I can't seem to pinpoint the issues on the CI. I've updated the YAML file to use the .net 7.0 SDK.

poppastring commented 1 year ago

Thanks for the PR! I will take a look at this, if I remember there is something I hardcoded that needs to be updated (I am sure it is the wrong way to do it).

To be honest, I tend to avoid STS version of .NET just because of the churn, however, there are some really compelling updates around rate limiting that I am looking forward to integrate.

I will also say that we are completing some larger changes that will ensure dasblog sits within the fediverse a bit more comfortably so expect a delay in the merge.

poppastring commented 1 year ago

I have decided to wait for LTS version of .NET before taking this on.