Skip to content
This repository was archived by the owner on Dec 24, 2025. It is now read-only.

Fix: multipart api improvement#13

Merged
CallMeEchoCodes merged 2 commits intoSpiritGameStudios:mainfrom
ekulxam:main
Jun 16, 2025
Merged

Fix: multipart api improvement#13
CallMeEchoCodes merged 2 commits intoSpiritGameStudios:mainfrom
ekulxam:main

Conversation

@ekulxam
Copy link
Copy Markdown
Collaborator

@ekulxam ekulxam commented Jun 14, 2025

Changes:

  • Renamed PartHolder#parts to PartHolder#getEntityParts
  • Renamed EntityPart.dims to EntityPart.dimensions and made it protected
  • Made EntityPart.owner protected
  • Overrode getDimensions in EntityPart
  • Slight mixin changes
  • EntityPartWorld is now an injected interface

Renamed `PartHolder#parts` to `PartHolder#getEntityParts`
Renamed `EntityPart.dims` to `EntityPart.dimensions` and made it protected
Made `EntityPart.owner` protected
Overrode `getDimensions` in `EntityPart`
Slight mixin changes
`EntityPartWorld` is now an injected interface
@CallMeEchoCodes CallMeEchoCodes self-requested a review June 14, 2025 06:43
Copy link
Copy Markdown
Member

@CallMeEchoCodes CallMeEchoCodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly good, just one mixin with too broad of an inject

@CallMeEchoCodes CallMeEchoCodes merged commit 8281efb into SpiritGameStudios:main Jun 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants