Skip to content

Fix several sound related bugs #425

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

Merged
merged 5 commits into from
Jul 20, 2025

Conversation

MicrocontrollersDev
Copy link
Contributor

@MicrocontrollersDev MicrocontrollersDev commented Jul 16, 2025

@MicrocontrollersDev
Copy link
Contributor Author

MicrocontrollersDev commented Jul 19, 2025

i am now unsure if we would want MC-248223 to be fixed. realistically in the default pack you cant tell, and pack creators can fix this themselves. if any custom server pack uses these sounds in wildly different ways, it could be weird. i'll remove it from this PR, let me know if you would still be interested in it though

Copy link
Owner

@isXander isXander left a comment

Choose a reason for hiding this comment

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

There are 4 mixins here but you claim in PATCHED.md to fix 5.
If a bug fix coincidentally fixes two bugs, this should be documented in a fix explanation and in JavaDoc.

@MicrocontrollersDev
Copy link
Contributor Author

There are 4 mixins here but you claim in PATCHED.md to fix 5. If a bug fix coincidentally fixes two bugs, this should be documented in a fix explanation and in JavaDoc.

forgot to remove it when i removed the fix for MC-248223, removed it now 👍

@isXander isXander merged commit 0acba63 into isXander:1.21.7 Jul 20, 2025
1 check passed
@MicrocontrollersDev MicrocontrollersDev deleted the sound-source-fixes branch July 20, 2025 20:08
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.

2 participants