Skip to content

Re-add shear methods from bukkit Shearable#11714

Closed
HaHaWTH wants to merge 1 commit intoPaperMC:masterfrom
HaHaWTH:master
Closed

Re-add shear methods from bukkit Shearable#11714
HaHaWTH wants to merge 1 commit intoPaperMC:masterfrom
HaHaWTH:master

Conversation

@HaHaWTH
Copy link
Contributor

@HaHaWTH HaHaWTH commented Dec 6, 2024

This pull request closes #11638.

Changes made

Readd shear related methods to Sheep and Bogged from bukkit Shearable.

This is my first PR to Paper, sorry if i did something wrong.

@HaHaWTH HaHaWTH requested a review from a team as a code owner December 6, 2024 20:58
@github-actions
Copy link

github-actions bot commented Dec 6, 2024

Please do not open pull requests from the master branch, create a new branch instead.

@github-actions github-actions bot closed this Dec 6, 2024
@electronicboy
Copy link
Member

We should expose what "sheared" represents, rather then just expanding the usage of flawed API; (and that should be added to the existing entity API patch)

@HaHaWTH
Copy link
Contributor Author

HaHaWTH commented Dec 6, 2024

We should expose what "sheared" represents, rather then just expanding the usage of flawed API; (and that should be added to the existing entity API patch)

Thanks for explanation! Should I add this to Missing Entity API patch?

@electronicboy
Copy link
Member

Yes

@electronicboy
Copy link
Member

Oh, actually, both of them called the internal data shreared, so, this is fine; Snowmen already have their "derp"'d methods

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

Sheep/Bogged/Snowman "regrow wool/mushroom/pumpkin" methods

2 participants