-
-
Notifications
You must be signed in to change notification settings - Fork 343
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
VillagerFactory redundant #337
Comments
AIOOB is right here, the villager factory has only methods that are already covered by other classes/interfaces so it should be removed. |
If we are to remove VillagerFactory, then LivingMetaFactory would be removed as well, just in case you haven't considered that ramification as well. |
I dont see any benefits for this class/interface either. |
This was fixed in e8c307a |
From what I can see the VillagerFactory class is useless as all the functionality it provides is also provided by GameRegistry and Career.
The text was updated successfully, but these errors were encountered: