-
Notifications
You must be signed in to change notification settings - Fork 7
Production Release #308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Production Release #308
Conversation
BlakeMScurr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR prepares a production release by introducing a new capability to rearrange a player's hand and updating related client and server functionality, along with database migration and dependency version bumps.
- Added RearrangeHand functionality to both server and client code.
- Updated migration scripts for changelogs.
- Upgraded the wasm-bindgen-cli version in the Dockerfile and GitHub workflow.
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| truncate_server/src/main.rs | Added a new match arm for RearrangeHand with a TODO for error handling. |
| truncate_server/src/game_state.rs | Added rearrange_hand method that currently ignores error results. |
| truncate_core/src/messages.rs | Introduced a new PlayerMessage::RearrangeHand variant. |
| truncate_client/src/regions/single_player.rs | Integrated rearrange_hand handling upon receiving the corresponding message. |
| truncate_client/src/lil_bits/hand.rs | Extended hand rendering logic for rearrangement with potential panic risk. |
| Other migration and dependency files | Updated migration scripts and dependency versions accordingly. |
Install binaryen from GitHub rather than apt
No description provided.