Skip to content

black rails#52

Merged
lukacslacko merged 3 commits into
mainfrom
black-rails
Jun 27, 2025
Merged

black rails#52
lukacslacko merged 3 commits into
mainfrom
black-rails

Conversation

@lukacslacko

Copy link
Copy Markdown
Owner

No description provided.

@lukacslacko lukacslacko requested review from Copilot and papjuli June 15, 2025 14:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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, titled "black rails," refactors rail and train handling to support dynamic path updates through improved entity creation, transformation, and state tracking.

  • Modified rail entity creation and transformation updates in src/ui.rs.
  • Updated Train component and its update method in src/train.rs to include State, Commands, and Assets changes.
  • Augmented the RailInfo struct in src/state.rs with an atomic counter for usage tracking.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/ui.rs Revised rail entity handling; now creating or reusing entities and initializing new fields.
src/train.rs Updated the Train component to tie transforms with rails and enhanced update logic with additional state and material handling.
src/state.rs Extended the RailInfo struct by adding an atomic counter field for tracking rail usage.

Comment thread src/ui.rs
Comment thread src/train.rs Outdated
@lukacslacko lukacslacko merged commit 8e873b3 into main Jun 27, 2025
1 check failed
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.

3 participants