use System.Configuration.ConfigurationManager 8.0, #2284
Conversation
as it has less dependencies than previous versions fixes dotnet#2232
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2284 +/- ##
==========================================
- Coverage 72.64% 72.47% -0.17%
==========================================
Files 310 310
Lines 62045 62045
==========================================
- Hits 45070 44965 -105
- Misses 16975 17080 +105
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
We are trying not to make major version dependency changes (including transitive) within non-major version bumps of MDS. So we'll merge this after the 5.2 GA. We'll bump to MDS 6.0 and include this change. It would be a nice refactor if we could have different dependency versions for the different target frameworks. I think that would allow us to make changes like this without breaking others. For example, adding future .NET 10 support with different dependency versions without a MDS major version bump and without breaking apps still targeting .NET 8. |
|
@David-Engel Ah - so that is why the .NET 8 dependencies are staying at 6.x (for 5.2 at least) I will see if I can find an approach to grouping the dependencies better - it already happens in the .nuspec file. |
|
Superseeded by #2303 |
as it has less dependencies than previous versions
fixes #2232