PeachCloud/peach-workspace
3
2
Fork 1
Code Issues 17 Pull Requests 1 Projects Releases Wiki Activity

Remove all instances of unwrap() and expect() from peach-lib #35

New Issue
Closed
opened 2021-11-23 13:58:16 +00:00 by glyph · 1 comment
glyph commented 2021-11-23 13:58:16 +00:00
Owner
Copy Link

I just noticed a few instances of unwrap() and expect() in peach-lib.

We need to handle those errors in a way which doesn't cause the calling application to panic.

I just noticed a few instances of `unwrap()` and `expect()` in peach-lib. We need to handle those errors in a way which doesn't cause the calling application to panic.
glyph added the
maintenance
 label 2021-11-23 13:58:16 +00:00
glyph commented 2021-12-07 08:01:21 +00:00
Author
Owner
Copy Link

Dealt with in PR#38.

Dealt with in [PR#38](https://git.coopcloud.tech/PeachCloud/peach-workspace/pulls/38).
glyph closed this issue 2021-12-07 08:01:21 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
documentation
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
maintenance
Pruning and weeding
peach-lib
peach-network
peach-oled
peach-stats
peach-web
question
More information is needed
refactor
Code rewrite
wontfix
This won't be fixed
No Label
maintenance
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
cblgh glyph notplants
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: PeachCloud/peach-workspace#35
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?