-
|
The latest release has a breaking change where ILogger has been removed from RulesEngine and all its constructors. The documentation still shows the requirement of the 3rd parameter. Before we refactor our implementation, I wanted to be sure that the removal of ILogger from the constructor is intentional and is the correct constructor signature going forward. |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 1 reply
-
|
Hi @ivansager, the removal of ILogger is intentional and is documented in https://github.com/microsoft/RulesEngine/blob/main/CHANGELOG.md It looks like there are some places in documentation which still needs updating. I will have a look at the documentation where it might be wrong |
Beta Was this translation helpful? Give feedback.
-
|
As for the reason of removal, ILogger was not used much internally, and we wanted to be consistent on how we return error/information. |
Beta Was this translation helpful? Give feedback.
Hi @ivansager, the removal of ILogger is intentional and is documented in https://github.com/microsoft/RulesEngine/blob/main/CHANGELOG.md
It looks like there are some places in documentation which still needs updating. I will have a look at the documentation where it might be wrong