-
Notifications
You must be signed in to change notification settings - Fork 22
Labels
good first issueGood for newcomersGood for newcomerstype: bugSomething isn't workingSomething isn't working
Description
Observed behavior
The InMemory provider is throwing an exception here:
| throw new TypeMismatchException($"flag {flagKey} is not of type {typeof(T)}"); |
We should move from throwing exceptions to returning an ErrorType with a default value. Something like:
return new ResolutionDetails<T>(flagKey, defaultValue, ErrorType.TypeMismatch, Reason.Error, $"flag {flagKey} is not of type {typeof(T)}");Expected Behavior
The In Memory provider doesn't throw any exceptions, and all unit tests pass.
Steps to reproduce
N/A
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomerstype: bugSomething isn't workingSomething isn't working