Skip to content

Start of new machine trait system#4479

Merged
gustovafing merged 29 commits into1.20.1-v8.0.0from
gus/machine-refactor-pt2
Jan 26, 2026
Merged

Start of new machine trait system#4479
gustovafing merged 29 commits into1.20.1-v8.0.0from
gus/machine-refactor-pt2

Conversation

@gustovafing
Copy link
Copy Markdown
Member

What

Initial impl of the new machine trait system.

@gustovafing gustovafing requested a review from a team as a code owner January 16, 2026 00:32
@github-actions github-actions bot added 1.20.1 Tests: Failed Game Tests have failed on this PR labels Jan 16, 2026
@github-actions github-actions bot added Tests: Passed Game Tests have passed on this PR and removed Tests: Failed Game Tests have failed on this PR labels Jan 18, 2026
Copy link
Copy Markdown
Contributor

@jurrejelle jurrejelle left a comment

Choose a reason for hiding this comment

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

thoughts on adding a getAllTraits() on the top level MetaMachine that's just a shortcut for getTraitHolder().getAllTraits()? I could see arguments for or against so just curious, since we'll use that method a lot.

@gustovafing gustovafing merged commit a8d3162 into 1.20.1-v8.0.0 Jan 26, 2026
4 checks passed
@gustovafing gustovafing deleted the gus/machine-refactor-pt2 branch January 26, 2026 07:29
@gustovafing gustovafing restored the gus/machine-refactor-pt2 branch January 26, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Tests: Passed Game Tests have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants