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

Heroes support #8

Merged
merged 4 commits into from
Apr 17, 2014
Merged

Heroes support #8

merged 4 commits into from
Apr 17, 2014

Conversation

ErgoAsh
Copy link
Contributor

@ErgoAsh ErgoAsh commented Mar 17, 2014

It was hard but I did it. It not direct compatibility with Heroes but it also give protect for "strange" potions effect, like Heroes skills or (maybe) McMMO skills and other plugins.

Sorry for those unreal additions ad deletes, EGit don't like GitHub. I have added only "skills" to config.yml, SuppressWarnings annotations and change effect of DamageCause.Magic by using try-catch.

I hope that pull request will repair a lot of bugs with other plugins.

It still need a lot of work in PlayerListener.java
It was hard but I did it. It not direct compatibility with Heroes but it
also give protect for "strange" potions effect, like Heroes skills or
(maybe) McMMO skills.
@ErgoAsh ErgoAsh mentioned this pull request Mar 17, 2014
@triarry
Copy link
Owner

triarry commented Apr 17, 2014

Sorry, I haven't had time to look at this. I'll merge and I'll fix the other issues that exist within the plugin.

triarry added a commit that referenced this pull request Apr 17, 2014
@triarry triarry merged commit 143ad66 into triarry:master Apr 17, 2014
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