search

asulwer / RulesEngine

Rules Engine with extensive Dynamic expression support
MIT License
26 stars 1 forks source link

add coverage exclusion for json converter #84

Closed JasonBoggsAtHMSdotCom closed 2 weeks ago

JasonBoggsAtHMSdotCom commented 2 weeks ago

Coverage is at 97% now

RenanCarlosPereira commented 2 weeks ago

hey @JasonBoggsAtHMSdotCom I dont think we should ignore the tests. specially for this class, I will open another PT, so you can check the tests.

thanks

JasonBoggsAtHMSdotCom commented 2 weeks ago

I chose to ignore this class since the converter is called by System.Text.Json and it's never called by RulesEngine directly

JasonBoggsAtHMSdotCom commented 2 weeks ago

Closing in favor of https://github.com/asulwer/RulesEngine/pull/85