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

Implement Villager Career API #563

Merged
merged 3 commits into from Sep 10, 2017

Conversation

Projects
None yet
2 participants
@momothereal
Member

momothereal commented Sep 10, 2017

Also includes I/O for some more Villager features:

  • Riches (something to do with traded emeralds)
  • Recipes (I/O)
Implement Villager Career API
Also includes I/O for some more Villager features.
@mastercoms

This comment has been minimized.

Show comment
Hide comment
@mastercoms

mastercoms Sep 10, 2017

Member

Code looks good, need to test though.

Member

mastercoms commented Sep 10, 2017

Code looks good, need to test though.

@momothereal

This comment has been minimized.

Show comment
Hide comment
@momothereal

momothereal Sep 10, 2017

Member

I have tested this, seems to work as intended.

Member

momothereal commented Sep 10, 2017

I have tested this, seems to work as intended.

@mastercoms mastercoms merged commit d5332e0 into GlowstoneMC:master Sep 10, 2017

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment