rollbar / Rollbar.NET

Rollbar for .NET
https://docs.rollbar.com/docs/dotnet
MIT License
65 stars 44 forks source link

Remove unnecessary explicit dependencies from the core Rollbar module and two new optional SDK modules #578

Closed akornich closed 3 years ago

akornich commented 3 years ago

Description of the change

  • resolve GH#577 - Remove unnecessary explicit dependencies from the core Rollbar module
  • resolve GH#576 - Move appsettings.json support into its own new module Rollbar.AppSettings.Json
  • resolve GH#575 - Move app.config support into its own new module Rollbar.App.Config

Namespace Rollbar.NetFramework is renamed into Rollbar.App.Config and moved to the similarly named new optional SDK module.

Namespace Rollbar.NetCore is renamed into Rollbar.AppSettings.Json and moved to the similarly named new optional SDK module.

A few unit tests moved to new dedicated unit test projects. A couple of new unit tests added.

It also includes 2 new sample apps to separately demonstrate the use of each of the new SDK modules. Some of the existing samples had to be updated to adapt to this refactoring effort. Some small code cleanup as needed while refactoring.

Type of change

Related issues

Implement #577 Implement #576 Implement #575

Checklists

Development

Code review

akornich commented 3 years ago

@bishopb , thanks for the prompt review!