Skip to content

Fix solutions for 05_ticket_v2\01_enum#268

Open
Wenxing-hou wants to merge 5 commits into
mainmatter:solutionsfrom
Wenxing-hou:fix_solution
Open

Fix solutions for 05_ticket_v2\01_enum#268
Wenxing-hou wants to merge 5 commits into
mainmatter:solutionsfrom
Wenxing-hou:fix_solution

Conversation

@Wenxing-hou

Copy link
Copy Markdown

Refer to the main code:

if status != "To-Do" && status != "In Progress" && status != "Done" {
panic!("Only `To-Do`, `In Progress`, and `Done` statuses are allowed");
}
Ticket {
title,
description,
status,
}

It is better to fix the check rather than delete it.

Signed-off-by: Wenxing Hou houwenxing@xiaomi.com

LukeMathWalker and others added 5 commits December 18, 2024 17:41
match panic message with the test expected message
It is better to fix the check rather than delete it.

Signed-off-by: Wenxing Hou <houwenxing@xiaomi.com>
@hdoordt
hdoordt force-pushed the solutions branch 2 times, most recently from 8c44511 to a46a8ae Compare June 15, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants