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

api: Make contains check for entity name #362

Merged
merged 3 commits into from May 23, 2021

Conversation

kezz
Copy link
Member

@kezz kezz commented May 12, 2021

This PR modifies Component#contains(Component) to check for the name of an entity in a ShowEntity hover event and adds some tests.

@kezz kezz force-pushed the make-contains-check-entities branch from 042b886 to 192de12 Compare May 17, 2021 11:54
@kezz
Copy link
Member Author

kezz commented May 17, 2021

Rebased following merge of #364.

@kezz kezz force-pushed the make-contains-check-entities branch from 192de12 to 506c3dc Compare May 17, 2021 17:30
@kashike kashike added this to Review in progress in Adventure May 19, 2021
@kezz kezz force-pushed the make-contains-check-entities branch from 506c3dc to abbbb63 Compare May 19, 2021 11:12
Adventure automation moved this from Review in progress to Reviewer approved May 21, 2021
@kashike kashike added this to the 4.8.0 milestone May 23, 2021
@kashike kashike merged commit 2b7eb00 into KyoriPowered:master May 23, 2021
Adventure automation moved this from Reviewer approved to Done May 23, 2021
@kezz kezz deleted the make-contains-check-entities branch May 25, 2021 10:07
@kashike kashike self-assigned this Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants