phylum-dev/vuln-reach

Move from assertions to `Result`

Closed this issue · 0 comments

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 Results in place of assertions.