Skip to content

Update to snapshot 25w09a & Improve Rendering Code#831

Merged
Prospector merged 8 commits intoTerraformersMC:1.21.5from
lowercasebtw:1.21.5
Mar 11, 2025
Merged

Update to snapshot 25w09a & Improve Rendering Code#831
Prospector merged 8 commits intoTerraformersMC:1.21.5from
lowercasebtw:1.21.5

Conversation

@lowercasebtw
Copy link
Copy Markdown
Contributor

@lowercasebtw lowercasebtw commented Feb 20, 2025

I improved how the rendering code was to use modern uses of RenderTypes and used VertexBuffer how it was before (in BufferUploader), and updated to 25w08a.

Also upgraded Loom, Gradle, & Fabric API.

Comment thread src/main/java/com/terraformersmc/modmenu/util/mod/fabric/FabricIconHandler.java Outdated
@lowercasebtw lowercasebtw changed the title Update to snapshot 25w08a & Improve Rendering Code Update to snapshot 25w09a & Improve Rendering Code Feb 27, 2025
Copy link
Copy Markdown
Member

@Prospector Prospector left a comment

Choose a reason for hiding this comment

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

This PR also seems to include a reformatting. I don't particularly mind, but it'd be great if you could include an editorconfig file or something so that we don't keep reformatting to a new person's formatter settings each time someone new is trying to contribute.

Comment thread src/main/java/com/terraformersmc/modmenu/api/UpdateInfo.java Outdated
@lowercasebtw
Copy link
Copy Markdown
Contributor Author

About the editor config thing, how would I do that?

@Prospector
Copy link
Copy Markdown
Member

image In IntelliJ at least you can export it here in settings under Code Style, and then it just goes in the root of the repo. Though, I guess I'm not 100% all the things that can be controlled by this. so not entirely sure if it'll work

@lowercasebtw
Copy link
Copy Markdown
Contributor Author

mk

@lowercasebtw
Copy link
Copy Markdown
Contributor Author

Added

@Prospector Prospector merged commit c41200e into TerraformersMC:1.21.5 Mar 11, 2025
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.

3 participants