refactor(tetris, space-invaders): migrate to Cougr ECS as core archit…#139
refactor(tetris, space-invaders): migrate to Cougr ECS as core archit…#139victor62-art wants to merge 3 commits into
Conversation
…ecture - Replace decorative/transitional ECS usage with real SimpleWorld ownership - Add components.rs and systems.rs to both examples - Remove game_state.rs from space_invaders (replaced by components + systems) - All game entities now live in SimpleWorld; systems query/mutate via ComponentTrait - GameState/GameMeta structs hold only non-entity metadata (score, level, etc.) - Remove all legacy, transitional, and old-version comments - Update README architecture sections for both examples - All existing tests pass: tetris 7/7, space_invaders 13/13
|
@victor62-art Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
Hey @victor62-art , please fix the workflow errors |
|
Hey @victor62-art , what are the updates on this pull request? Reminder: The abandoned pr will be closed and the issue will be reassigned. |
|
@salazarsebas yh i'll fix it now |
|
@salazarsebas sorry for taking this long but i'm done |
|
Hey @victor62-art , the workflows keep failing because you haven't run the |
|
@salazarsebas i've done that but it's not the issue |
According to the workflow logs, it says that the fmt fails |
|
The PR is closed because there were no updates |
3 participants
…ecture
closes #137