-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
feat: ActorSpawnManager.SpawnInterval supports 1 or 2 values #18794
feat: ActorSpawnManager.SpawnInterval supports 1 or 2 values #18794
Conversation
c0ec777
to
361f0fe
Compare
@@ -25,8 +25,8 @@ public class ActorSpawnManagerInfo : ConditionalTraitInfo, Requires<MapCreepsInf | |||
[Desc("Maximum number of actors.")] | |||
public readonly int Maximum = 4; | |||
|
|||
[Desc("Time (in ticks) between actor spawn.")] | |||
public readonly int SpawnInterval = 6000; | |||
[Desc("Time (in ticks) between actor spawn. Supports 1 or 2 values.\nIf 2 values are provided they are used as a range from which a value is randomly selected.")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The \n
is not strictly necessary, can remove it if we want.
361f0fe
to
c89661f
Compare
c89661f
to
1ef0eeb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes lgtm otherwise, and worms spawn as expected.
Providing 2 values creates a range from which a value is randomly selected.
1ef0eeb
to
ffed034
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
If merged this will close #18788.
Testcase: