Skip to content

Conversation

@JustAHuman-xD
Copy link
Contributor

@JustAHuman-xD JustAHuman-xD commented Sep 25, 2025

Relies on #268 (the relevant core pr changed item stack stuff so this ones core pr had to be based off of that ones)

Adds missing values & fixes some values for different equipment items while switching to using the new methods from PylonItemStackBuilder
see pylonmc/pylon-core#352

Fixes #260

@Seggan
Copy link
Member

Seggan commented Sep 28, 2025

This still a thing after closing the Core PR

@JustAHuman-xD
Copy link
Contributor Author

yes, im working on updating it lol

@JustAHuman-xD
Copy link
Contributor Author

theres a new core PR incoming

@JustAHuman-xD JustAHuman-xD changed the title [ref:armor-tool-builders] Equipment Update [ref:update-item-builders] Equipment Update Sep 28, 2025
@JustAHuman-xD JustAHuman-xD changed the base branch from master to use-vanilla-entities-in-pylon-entity-block-holder September 28, 2025 21:46
@JustAHuman-xD JustAHuman-xD marked this pull request as draft September 30, 2025 06:31
@JustAHuman-xD JustAHuman-xD marked this pull request as ready for review October 1, 2025 21:31
Base automatically changed from use-vanilla-entities-in-pylon-entity-block-holder to master October 4, 2025 16:43
Copy link
Contributor

@OhmV-IR OhmV-IR left a comment

Choose a reason for hiding this comment

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

lgtm cw & ig

Co-authored-by: Adrien Bourdeaux <[email protected]>
@LordIdra LordIdra merged commit 1903f8c into master Oct 13, 2025
2 checks passed
@LordIdra LordIdra deleted the fix-bronze-tools branch October 13, 2025 02:28
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.

Bronze tools not working as expected

5 participants