Skip to content

Fireproof Rune Fixes#158

Merged
LordIdra merged 7 commits intopylonmc:masterfrom
balugaq:fix/fireproof-rune
Aug 26, 2025
Merged

Fireproof Rune Fixes#158
LordIdra merged 7 commits intopylonmc:masterfrom
balugaq:fix/fireproof-rune

Conversation

@balugaq
Copy link
Collaborator

@balugaq balugaq commented Aug 23, 2025

Closes #155
Closes #156
Closes #157

Can't compile the code for the unknown code. Comment under this PR if issues still exist.
dd646520951524e7acd377649eeecb05

@Seggan
Copy link
Member

Seggan commented Aug 23, 2025

Ugh it didn't work

@balugaq
Copy link
Collaborator Author

balugaq commented Aug 23, 2025

What issues?

@Seggan
Copy link
Member

Seggan commented Aug 23, 2025

The failing CI

@balugaq
Copy link
Collaborator Author

balugaq commented Aug 23, 2025

Uhh, at least as of now, the CI checks for the several PRs I’ve submitted have all failed.

@Seggan
Copy link
Member

Seggan commented Aug 23, 2025

Yeah I am aware, it's a permissions thing. The dependency submission should really only be running on master anyway. I've tried again.

@Seggan
Copy link
Member

Seggan commented Aug 23, 2025

There ya go

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.

Fireproof rune no longer seems to work..?

@LordIdra
Copy link
Contributor

image oh

@LordIdra
Copy link
Contributor

Use ItemStackBuilder.lore to add lore lines instead of

.editData(DataComponentTypes.LORE, lore -> {
                    List<Component> lines = lore.lines();
                    lines.add(TOOLTIP);
                    return ItemLore.lore(lines);
                })

that should fix it

@LordIdra
Copy link
Contributor

I fixed it locally but now there are more errors in console. I'll leave you to have a look

@balugaq
Copy link
Collaborator Author

balugaq commented Aug 24, 2025

Fine, just send errors here.

mclo.gs

@LordIdra
Copy link
Contributor

Can you please test locally before PRing? You can do this with the master project, if you need help then let us know. It's much easier for you to test then for me to test and report lots of issues and then you fix them

@balugaq
Copy link
Collaborator Author

balugaq commented Aug 24, 2025

I'll test since I set up the master project just now.

@balugaq balugaq mentioned this pull request Aug 25, 2025
@balugaq balugaq requested a review from LordIdra August 25, 2025 12:48
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.

Nice, thanks for the PR!

@LordIdra LordIdra merged commit caa59c6 into pylonmc:master Aug 26, 2025
2 checks passed
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