Remove redundant return values from methods #381

Open
opened 2025-02-13 12:10:41 +00:00 by LunarTides · 1 comment
LunarTides commented 2025-02-13 12:10:41 +00:00 (Migrated from github.com)

A lot of methods in the Card and Player class returns a success boolean. Only keep these if they're actually used.

A lot of methods in the `Card` and `Player` class returns a success boolean. Only keep these if they're actually used.
LunarTides commented 2025-02-20 10:05:10 +00:00 (Migrated from github.com)

This needs extra care. I don't want to remove the return values if they can be useful at some point. We need to achieve some sort of balance.

This needs extra care. I don't want to remove the return values if they _can_ be useful at some point. We need to achieve some sort of balance.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
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
LunarTides/Hearthstone.js#381
No description provided.