Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

toss a lot of redundant mappings #630

Merged
merged 22 commits into from
Aug 28, 2024

Conversation

ix0rai
Copy link
Member

@ix0rai ix0rai commented Aug 20, 2024

a follow-up to #629

@ix0rai ix0rai added t: toolchain changes to the quilt mappings toolchain v: snapshot targets a snapshot version of minecraft reviews needed please review this PR s: medium PRs with less than 700 lines and more than 200 labels Aug 20, 2024
@@ -14,7 +14,6 @@ CLASS net/minecraft/unmapped/C_oddtxnnw net/minecraft/client/render/entity/Entit
ARG 2 tickDelta
METHOD m_gkfhbwdx getPositionOffset (Lnet/minecraft/unmapped/C_vptppxob;)Lnet/minecraft/unmapped/C_vgpupfxx;
METHOD m_hknkoqwr getTexture (Lnet/minecraft/unmapped/C_vptppxob;)Lnet/minecraft/unmapped/C_ncpywfca;
METHOD m_idqrsxfa (Lnet/minecraft/unmapped/C_vptppxob;)F
Copy link
Member

Choose a reason for hiding this comment

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

This is something to look into for the javadocs:
image

@ix0rai
Copy link
Member Author

ix0rai commented Aug 21, 2024

adding suggestions individually because the file tab is huge. sorry

ix0rai and others added 7 commits August 20, 2024 23:14
Co-authored-by: Will <supersaiyansubtlety@gmail.com>
Co-authored-by: Eli Orona <eliorona@live.com>
Co-authored-by: Eli Orona <eliorona@live.com>
Co-authored-by: Eli Orona <eliorona@live.com>
Co-authored-by: Eli Orona <eliorona@live.com>
Co-authored-by: Eli Orona <eliorona@live.com>
Co-authored-by: Will <supersaiyansubtlety@gmail.com>
@OroArmor OroArmor added the update-base used to notify github actions that the base branch should be updated label Aug 21, 2024
Copy link
Contributor

🚨 Target branch is already set to 24w33a

@github-actions github-actions bot removed the update-base used to notify github actions that the base branch should be updated label Aug 21, 2024
@OroArmor OroArmor added the update-base used to notify github actions that the base branch should be updated label Aug 21, 2024
Copy link
Contributor

🚀 Target branch has been updated to 24w34a

@github-actions github-actions bot changed the base branch from 24w33a to 24w34a August 21, 2024 23:18
Copy link
Contributor

🚨 Please fix merge conflicts before this can be merged

@github-actions github-actions bot added outdated this pull request hasn't been updated to the latest version or has merge conflicts and removed update-base used to notify github actions that the base branch should be updated labels Aug 21, 2024
…pings

# Conflicts:
#	mappings/com/mojang/blaze3d/platform/GlStateManager.mapping
#	mappings/com/mojang/blaze3d/shader/GlUniform.mapping
#	mappings/com/mojang/blaze3d/systems/RenderSystem.mapping
#	mappings/net/minecraft/client/gl/JsonEffectGlShader.mapping
#	mappings/net/minecraft/client/gl/PostProcessShader.mapping
#	mappings/net/minecraft/client/gl/ShaderEffect.mapping
#	mappings/net/minecraft/client/render/BufferBuilderStorage.mapping
#	mappings/net/minecraft/client/render/RenderLayer.mapping
#	mappings/net/minecraft/client/render/ShaderProgram.mapping
#	mappings/net/minecraft/world/storage/StorageIoWorker.mapping
@ix0rai ix0rai removed the outdated this pull request hasn't been updated to the latest version or has merge conflicts label Aug 23, 2024
Co-authored-by: Eli Orona <eliorona@live.com>
@OroArmor OroArmor added final-comment-period is approved and will soon be merged if no issues are raised and removed reviews needed please review this PR labels Aug 26, 2024
@OroArmor OroArmor merged commit c2895a0 into QuiltMC:24w34a Aug 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
final-comment-period is approved and will soon be merged if no issues are raised s: medium PRs with less than 700 lines and more than 200 t: toolchain changes to the quilt mappings toolchain v: snapshot targets a snapshot version of minecraft
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants