The switch to .NET 8.0 makes this change look bigger than it is. While upgrading the choku lib to .NET 8, I made a bunch of underlying breaking changes to its APIs (just renaming classes and such) which is why you see a bunch of changes throughout.
Highlights:
.NET 8 upgrade, including a bunch of small changes to code style that come with that upgrade to a newer C#
Upgrade a ton of packages along the way, including bumping NitroPacker and the Choku lib to their .NET 8 versions
Before, if a patch failed to apply, we left the build directory in an unclean state (with patch files just sitting there). New code explicitly reverts those patches and deletes unnecessary directories.
Added functionality to NitroPacker to allow naming the docker containers we create when patching, which gives us the handy ability to call docker rm afterwards and clean up those containers
Surfaced the make/docker logging to the user through a loading modal which will hopefully make that process less confusing
Closes #306. Closes #307. Fixes #308. Closes #311.
The switch to .NET 8.0 makes this change look bigger than it is. While upgrading the choku lib to .NET 8, I made a bunch of underlying breaking changes to its APIs (just renaming classes and such) which is why you see a bunch of changes throughout.
Highlights:
docker rmafterwards and clean up those containers