Move from assertions to `Result`
Closed this issue · 0 comments
andreaphylum commented
There are a few places in the codebase where assertions and unwraps are used in order to mark the failure to uphold invariants as outright bugs. #23 is an example of such a case.
We should review the code and figure out where it is appropriate to keep using assertions and where we could just want to skip processing, moving to use Result
s in place of assertions.