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

Update to 1.20.1 to support more entity types. #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

albatrossen
Copy link

This caused it to give an error on commons-lang missing, so added as a dependency and shaded it.

This caused it to give an error on commons-lang missing, so added
as a dependency and shaded it.
@2008Choco
Copy link
Owner

2008Choco commented Jan 31, 2024

I would rather just replace instances of Commons Lang method references. I haven't looked but it's probably just NumberUtils or WordUtils which can be very easily replaced.

@albatrossen
Copy link
Author

Understandable....I did consider doing that as well but went off with something really quick first...I might take a look later when I have more time.

@2008Choco
Copy link
Owner

I'm in no particular rush. This plugin has very much been on the backburner, but I would really like to get it updated. I've had a few people ask me. I'm just not sure if I have the time to maintain it every version. So again, take your time, I probably won't be able to actually make a release with many changes anyways for at least a few weeks :)

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.

2 participants