Skip to content
This repository has been archived by the owner on Jan 3, 2020. It is now read-only.

Make MrbFile::require fallible #25

Closed
lopopolo opened this issue May 17, 2019 · 0 comments
Closed

Make MrbFile::require fallible #25

lopopolo opened this issue May 17, 2019 · 0 comments
Labels
good first issue Good for newcomers

Comments

@lopopolo
Copy link
Member

There is currently a lot of unwrapping going on in MrbFile implementations. Make require return Result<(), MrbError> to propagate these errors to the HTTP layer.
@lopopolo lopopolo added the good first issue Good for newcomers label May 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
good first issue Good for newcomers
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lopopolo