Option Type: replace pattern matching by the corresponding syntaxic sugar #13

Merged
lx merged 2 commits from feature/syntaxic-sugar into master 2020-11-11 15:38:59 +00:00
Owner

Could you specifically check the modification in s3_list.rs that uses and_then and the modification in s3_put.rs that uses unwrap_or?
Other ones use simply the ok_or(..)? pattern

Could you specifically check the modification in `s3_list.rs` that uses `and_then` and the modification in `s3_put.rs` that uses `unwrap_or`? Other ones use simply the `ok_or(..)?` pattern
quentin changed title from Option Type: replace pattern matching by corresponding syntaxic sugar to Option Type: replace pattern matching by the corresponding syntaxic sugar 2020-11-11 15:14:25 +00:00
Owner

Approved

Approved
lx merged commit b00e29f931 into master 2020-11-11 15:38:58 +00:00
lx deleted branch feature/syntaxic-sugar 2020-11-11 15:39:08 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/garage#13
No description provided.