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

Tweaked goomba plugin support. #29

Closed
wants to merge 3 commits into from
Closed

Tweaked goomba plugin support. #29

wants to merge 3 commits into from

Conversation

WildCard65
Copy link
Contributor

Hope this pleases you Wliu(also I haven't touched the PR things, last one I touched the edit button of where pull request gets merged into but left them at default and last one had auto merge avail).
Also note: Recent commit from Wliu is marked as authored by me, making note that recent commit on official git side's are Wliu's: d7f0da7)

public Action:OnStomp(attacker, victim, &Float:damageMultiplier, &Float:damageBonus, &Float:JumpPower)
{
Debug("Stomp happening!");
if (!Enabled)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if(!Enabled || !IsValidClient(attacker) || !IsValidClient(victim))

@50Wliu
Copy link
Member

50Wliu commented Mar 27, 2014

Your PR is still unmergeable. Try force-resetting your branch to mine, so that there are no extra commits floating around, then re-submit.
If you know how to squash, you could do that too.

@WildCard65 WildCard65 closed this Mar 28, 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.

None yet

2 participants