Capgemini / xrm-datamigration

Export and import data for Microsoft Dataverse. Supports JSON and CSV.
MIT License
24 stars 11 forks source link

Refactor to use ILogger #28

Open tdashworth opened 3 years ago

tdashworth commented 3 years ago

Is your feature request related to a problem? Please describe. This isn't problem-related but while refactoring powerapps-packagedeployertemplate, I switch the services to use Microsoft's Logging Extensions. While integrating with this library, I wrote a wrapper for a very similar ILogger interface.

Describe the solution you'd like My question and the proposal is to update this library to use the same logging abstraction provided by MS for better integration with other tooling. For example, the respective Xrm Toolbox plugin could also be updated with a custom ILogger implementation.

Additional context https://docs.microsoft.com/en-us/dotnet/core/extensions/logging

I've opened this as a discussion before we consider investing time in the refactor. 😄