update Newtonsoft.Json to 13.0.3#168
Conversation
6a18410 to
2e60a49
Compare
|
Why did you update to this version ? We tend to keep satellite assemblies to the lowest version possible (unless vulnerabilities are discovered) to avoid conflicts for consumers (ie: they are using a package that references a lower version of Newtonsoft.JSON). Unless you find this is a bad pratice or useless, is it required ? |
|
Sorry for posting this PR without any comment. As far as I'm concerned, it's good practice to keep dependencies up to date, in order to include all the latest bugfixes and improvements. In this case, I'm not updating the package for a particular fix, but just to have the latest version (for good practice). I guess it's up to you, whether you want to merge this or not. It's probably not completely unreasonable to be a bit more conservative about the dependencies, and only update if it's really necessary (updates can potentially introduce new bugs and problems as well). |
2e60a49 to
a4e0021
Compare
|
In any case, I rebased it to current master now, in order to take advantage of the latest GHA enhancements. |
a4e0021 to
e931037
Compare
No description provided.