Update target frameworks & packages #388
Conversation
|
This seems to be a bit of a mass update throughout the code base by changing the targets and then just updating all nuget packages to their latest versions and some linting (Directory slash changes, Variable renaming, etc). Are there any regressions in any of the packages which were updated? Don't want to be rude, but the latest PRs you have been issuing lately appear to be less about fixing things in the code and rather increasing the number of commits you have. Delsin-Yu/GDTask.Nuget#7 |
Hi @DJGosnell. First of all, I'm not interested in "increasing the number of commits I have", and I don't think anyone cares. Is there a rule against contributing to too many open source projects? All of the listed repositories are being used by my various projects. If you don't want me to make pull requests to this repository, then I won't, that's fine. The goal of this PR was to upgrade all of the outdated dependencies in MemoryPack. .NET 6/7 are no longer installed by default since they're out-of-support, which was giving me warnings and errors, so I decided to try to bring everything up-to-date. I assumed the .NET 6 vs .NET 7 project was not really being used anymore but if not then I can undo that change. Please understand that I have the best intentions. |
|
No hard feelings here. I have just been on the wrong side of people trying to contribute for the sake of increasing their own stats. E.g. DigitalOcean Hacktoberfest. This is not my repository, but I use it extensively so I care about it. As long as things are in provided in good-faith, it is always great for the community. |
This pull request updates the target frameworks and packages of every project, including one vulnerable dependency and one beta dependency. MemoryPack drops
net7.0and addsnet9.0(as well as the existingnetstandard2.1andnet8.0).