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

Adds entity sound expression and test #7019

Merged
merged 11 commits into from
Oct 13, 2024

Conversation

cheeezburga
Copy link
Member

Description

This PR adds an expression to get a sound of a living entity. The gettable sounds are death, damage, swim, fall, high fall, fall from x distance, splash, speedy splash, eating, eating with x item, drinking, drinking with x item


Target Minecraft Versions: 1.19.2 and higher
Requirements: none
Related Issues: none

Co-authored-by: _tud <98935832+UnderscoreTud@users.noreply.github.com>
- Updates fall damage pattern to not conflict with new block sound PR
- Fixes some other patterns missing some small details
- Adds Skript doc annotations
- Adds new ambient sound
@cheeezburga cheeezburga added the feature Pull request adding a new feature. label Sep 22, 2024
Copy link
Member

@sovdeeth sovdeeth left a comment

Choose a reason for hiding this comment

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

you could probably combine height and item into a single object parameter but that might just be worse

seems good

@sovdeeth sovdeeth added the feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. label Sep 26, 2024
@sovdeeth sovdeeth merged commit 7cf5d79 into SkriptLang:dev/feature Oct 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull request adding a new feature. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants