arcus.observability
arcus.observability copied to clipboard
refactor: Removing all dependency to Guard.NET.
As per issue 577. This PR removes all dependency on Guard.NET. All checks done by Guard.NET have been replaced with if checks.
Closes #577
Deploy request for arcus-observability pending review.
Visit the deploys page to approve it
| Name | Link |
|---|---|
| Latest commit | 2a400193b0bac0d124daba38abca9766ec6b1d64 |
@stijnmoreels
We should use
ArgumentExceptionwhen checking for blank strings, notArgumentNullException.
Only when checking for blank strings?
@stijnmoreels
We should use
ArgumentExceptionwhen checking for blank strings, notArgumentNullException.Only when checking for blank strings?
We can fallback on ArgumentException for both null, "" and " " inputs.
I'm halfway through moving ArgumentNullExceptions to ArgumentException. I've also took the liberty of adding and adapting the following comments
Adapted all the code. Ready for another review