discord
/
hal-9000
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add randommk command #2

Merged
xyon merged 5 commits from awsxdr/hal-9000:mkrandomizer into main 2025-05-25 13:29:05 +00:00
Conversation 0 Commits 5 Files Changed 3 +378 -3
awsxdr commented 2025-05-25 12:32:10 +00:00
Contributor
Copy Link

Add a new command (/randommk) to HAL to allow the randomization of Mario Kart 8 Deluxe kart combinations.

Add a new command (`/randommk`) to HAL to allow the randomization of Mario Kart 8 Deluxe kart combinations.
awsxdr added 4 commits 2025-05-25 12:32:10 +00:00
xyon requested changes 2025-05-25 12:49:40 +00:00
src/hal/commands/mario_kart_randomizer.rs Outdated
@ -0,0 +1,334 @@
use std::collections::HashMap;
xyon commented 2025-05-25 12:49:24 +00:00
Owner
Copy Link

This looks like it belongs in models rather than commands, since it's a structure called on behalf of the command rather than the command itself

This looks like it belongs in models rather than commands, since it's a structure called on behalf of the command rather than the command itself
awsxdr commented 2025-05-25 13:05:29 +00:00
Author
Contributor
Copy Link

Fair. I wasn't sure where to put a struct that's basically just a collection of stateless methods. Moved in d31513d37b.

Fair. I wasn't sure where to put a struct that's basically just a collection of stateless methods. Moved in d31513d37bd3d4f5caa8cd5595ef2273c8cf70c5.
xyon marked this conversation as resolved
awsxdr added 1 commit 2025-05-25 13:05:06 +00:00
awsxdr requested review from xyon 2025-05-25 13:06:20 +00:00
xyon merged commit f8fd096c0f into main 2025-05-25 13:29:05 +00:00
xyon deleted branch mkrandomizer 2025-05-25 13:29:05 +00:00
xyon referenced this issue from a commit 2025-05-25 13:29:07 +00:00
Add randommk command (#2)
Sign in to join this conversation.
Reviewers
No reviewers
xyon
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: discord/hal-9000#2
No description provided.
Delete Branch "awsxdr/hal-9000:mkrandomizer"

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?