Skip to content

[ref:vaan/fix/tool-fixup] Fix Tools#810

Merged
LordIdra merged 4 commits intomasterfrom
vaan/fix/tool-fixup
Mar 11, 2026
Merged

[ref:vaan/fix/tool-fixup] Fix Tools#810
LordIdra merged 4 commits intomasterfrom
vaan/fix/tool-fixup

Conversation

@Intybyte
Copy link
Member

@Intybyte Intybyte commented Mar 1, 2026

Fixes #697
Fixes #792
Fixes #790
Fixes #789
Fixes #788

@Intybyte Intybyte requested a review from a team March 6, 2026 11:37
Copy link
Member

@Seggan Seggan left a comment

Choose a reason for hiding this comment

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

You never seem to actually use any of the item settings anywhere?

@LordIdra
Copy link
Contributor

LordIdra commented Mar 9, 2026

You never seem to actually use any of the item settings anywhere?

rebarToolWeapon uses them implicitly

@Seggan
Copy link
Member

Seggan commented Mar 10, 2026

Does not compile with latest Rebar, cannot test

@Intybyte
Copy link
Member Author

Does not compile with latest Rebar, cannot test

Did a merge, now it compiles if it didn't before

Copy link
Contributor

@LordIdra LordIdra left a comment

Choose a reason for hiding this comment

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

All issue fixes tested

@LordIdra LordIdra merged commit 5349f0a into master Mar 11, 2026
1 of 2 checks passed
@LordIdra LordIdra deleted the vaan/fix/tool-fixup branch March 11, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants