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

Removed broken reimplementation of Divisor's old PlayerAPI. #139

Merged
merged 2 commits into from
Dec 28, 2024

Conversation

mineLdiver
Copy link
Member

Also removed CustomArmorValue because, as far as I know, it wasn't used in any mod, and the inaccuracy of damage calculation wasn't worth it.

It's very possible to reimplement it in a different form, but that should be thought through separately (do we want to calculate after vanilla calculations, before, or somehow in the middle? do we work with damage negation, or actual damage dealt (because those are very different things)?).

@mineLdiver mineLdiver added the enhancement New feature or request label Dec 23, 2024
@mineLdiver
Copy link
Member Author

Will close #135 and #83

@mineLdiver mineLdiver merged commit 4949fcc into develop Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant