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

Port gdi02 to New Lua #6324

Merged
merged 1 commit into from
Aug 31, 2014
Merged

Port gdi02 to New Lua #6324

merged 1 commit into from
Aug 31, 2014

Conversation

obrakmann
Copy link
Contributor

No description provided.

@Mailaender
Copy link
Member

The power plant does not get destroyed anymore:

image

Not sure what changed the balancing here.

@obrakmann
Copy link
Contributor Author

It doesn't in -release either. Did it before?

@Mailaender
Copy link
Member

Yes, it did it when I created/imported it and that was the way it was designed. http://youtu.be/5uGjJV9jnLg?t=12m8s

@Mailaender
Copy link
Member

Unrelated to these changes, but it still would be nice if this could be fixed in one go.

@obrakmann
Copy link
Contributor Author

updated

@Mailaender
Copy link
Member

👍

local attackers = Utils.Skip(nodUnits, #nodUnits - AttackerSquadSize)
Utils.Do(attackers, function(unit)
unit.AttackMove(waypoint2.Location)
Trigger.OnIdle(unit, function() unit.Hunt() end)
Copy link
Member

Choose a reason for hiding this comment

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

Trigger.OnIdle(unit, unit.Hunt) should work in cases like this. You only need to wrap in a function if the argument count/order are different.

@pchote
Copy link
Member

pchote commented Aug 31, 2014

:+2: with those fixes.

@obrakmann
Copy link
Contributor Author

updated

pchote added a commit that referenced this pull request Aug 31, 2014
@pchote pchote merged commit 9ed9096 into OpenRA:bleed Aug 31, 2014
@obrakmann obrakmann deleted the gdi02 branch September 3, 2014 20:31
@obrakmann obrakmann mentioned this pull request Sep 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants