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

Added Spawn Egg Functionality #350

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

OfficialKris
Copy link
Contributor

Description

Added entities.json and extractor
Added entities registry
Added support for right clicking spawn egg and spawning entity
Added add_mob function in server (very rough, probably can be put into "world")

Kept Entities_Registry Enum as it is used in many places, probably can be substituted for macro

Testing

Tested both block placing and egg spawning
Tested different types of eggs and all worked
Future work for separate PRs:

  • Interacting with mobs causes server to kick player because only attacks between players are allowed.
  • Unique characteristics are not supported (horses color, slime size)
  • Mob ai

Checklist

Things need to be done before this Pull Request can be merged.

  • Code is well-formatted and adheres to project style guidelines: cargo fmt
  • Code does not produce any clippy warnings: cargo clippy
  • All unit tests pass: cargo test
  • I added new unit tests, so other people don't accidentally break my code by changing other parts of the codebase. How?

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.

1 participant