sorceress/rustcord
sorceress
/
rustcord
1
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

beeeeg refactor :3 #1

Merged
sorceress merged 0 commits from videogame-hacker/rustcord:patch/cleanup-match-statements into master 2021-07-08 15:42:24 +00:00
Conversation 1 Commits - Files Changed 5 +122 -151
videogame-hacker commented 2021-07-07 23:24:59 +00:00
Contributor
Copy Link

i like using map_err and ? instead of big match statements

we also try to keep the 'happy path' unindented in this PR :>

i like using `map_err` and `?` instead of big match statements we also try to keep the 'happy path' unindented in this PR :>
videogame-hacker added 7 commits 2021-07-07 23:24:59 +00:00
aecdc7a273 Switch 'match' to 'if let' in main.rs::after 
This lets us drop a level of indentation, and remove the clunky match arm for Ok(()) => ()

:)
sorceress commented 2021-07-08 15:41:57 +00:00
Owner
Copy Link

thancc u

thancc u
sorceress merged commit d7f7014404 into master 2021-07-08 15:42:24 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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: sorceress/rustcord#1
No description provided.
Delete Branch "videogame-hacker/rustcord:patch/cleanup-match-statements"

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?