Page MenuHomeWildfire Games

Allow buildings to autobuild themselves
ClosedPublic

Authored by Stan on Dec 21 2019, 12:15 PM.

Details

Reviewers
Silier
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP23514: Allow buildings to autobuild themselves.
Summary

Currently you need to have builders to build buildings. This may sound logical, but consider buildings as living things, then they should grow by themselves.

This patch adds an autobuild flag, so that a placed foundation can build itself.

Question. This currently uses the builder component, meaning that such entity must also have one. This is done to remove duplication but it increases coupling. Should it be separated.

Test Plan

Add

<Buildable>
 <AutoBuild>
   <Rate>1</Rate>
 </AutoBuild>
</Buildable>

to a building of your choice, place its foundation, and check it builds by itself

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
Stan added a comment.Feb 19 2020, 8:11 AM

Invert condition check for rate !=0

Stan updated this revision to Diff 11394.Feb 19 2020, 8:12 AM
Stan marked 9 inline comments as done.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1789/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1790/display/redirect

Silier requested changes to this revision.Feb 19 2020, 7:18 PM

Autobuildable needs to watch for OnOwnershipChanged message and recalculate this.rate.
Else looks acceptable.

This revision now requires changes to proceed.Feb 19 2020, 7:18 PM

almost missed

binaries/data/mods/public/simulation/components/AutoBuildable.js
5

this.rate = applyvaluemodif...

Stan updated this revision to Diff 11395.Feb 19 2020, 8:41 PM
Stan marked an inline comment as done.

Listen to onownershipchanged message

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1791/display/redirect

For me and future me: Why is it a bad idea to have the Autobuild timer in the Autobuild component?

Silier requested changes to this revision.Feb 20 2020, 7:51 PM

when rate drops to 0, gui shows remaining time but that is not correct information,
(maybe some autobuild icon or tooltip could be added as well)

This revision now requires changes to proceed.Feb 20 2020, 7:51 PM
Stan added a comment.Feb 20 2020, 9:39 PM

For me and future me: Why is it a bad idea to have the Autobuild timer in the Autobuild component?

It's not bad, it's not doable, because the foundation creates a preview entity, and that's the one you add builders to. The problem here is that one cannot access that preview entity.

But I sent you a paste last week which had the timer in the Autobuildable cmp. and that worked?

Stan updated this revision to Diff 11406.Feb 20 2020, 9:53 PM
  • Cancel timers when rate is 0

But I sent you a paste last week which had the timer in the Autobuildable cmp. and that worked?

I don't know then, but the thing I tried failed, and if that paste isn't changing the preview entity, then it's changing behavior :)

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1798/display/redirect

Stan updated this revision to Diff 11408.Feb 20 2020, 11:11 PM

Fix broken check.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1800/display/redirect

Silier requested changes to this revision.Feb 21 2020, 7:51 AM
Silier added inline comments.
binaries/data/mods/public/simulation/components/Foundation.js
523

this.CancelTimer means timer is running and we want to cancel it, right ? But if timer is running L518 executes early return so never gets here, also AutoBuild will cancel timer if rate reaches 0

525

even if you cancel timer above when rate reaches 0, this line starts it again

This revision now requires changes to proceed.Feb 21 2020, 7:51 AM
Stan updated this revision to Diff 11409.Feb 21 2020, 8:28 AM

Remove cancelling

Stan marked 2 inline comments as done.Feb 21 2020, 8:29 AM

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1801/display/redirect

part of autobuildable has class syntax and part prototype, should use one

binaries/data/mods/public/simulation/components/Foundation.js
506

maybe !rate

Stan added a comment.Feb 22 2020, 12:02 AM

Have to use both. Events are not supported in class syntax.

Stan marked an inline comment as done.Feb 22 2020, 10:05 AM
Stan added inline comments.
binaries/data/mods/public/simulation/components/Foundation.js
506

I'd rather not, as it would cover rate being negative.

Silier accepted this revision.Feb 23 2020, 10:48 AM

Structure with autobuildable component builds without unit building it.
Modifications from technologies and auras apply correctly.

Feature requested for HC.

binaries/data/mods/public/simulation/components/Foundation.js
506

!-5 == false

This revision is now accepted and ready to land.Feb 23 2020, 10:48 AM
Stan marked an inline comment as done.EditedFeb 23 2020, 12:16 PM

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)

Stan` shoots himself in the foot: Shouldn't this use Status Effects? Could be done in V2.

Thanks for the review Angen.

binaries/data/mods/public/simulation/components/Foundation.js
506

@Angen should I change it then?

Silier added inline comments.Feb 23 2020, 5:16 PM
binaries/data/mods/public/simulation/components/Foundation.js
506

its not terrible or bad as it is so i leave that on you to decide and rate will always be defined

In D2492#110907, @Stan wrote:

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)


(You'd need to update the Foundation test though.)

In D2492#110907, @Stan wrote:

Stan` shoots himself in the foot: Shouldn't this use Status Effects? Could be done in V2.

I'm not sure how one could use Status Effects for this?

Stan added a comment.Feb 24 2020, 8:24 PM
In D2492#110907, @Stan wrote:

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)


(You'd need to update the Foundation test though.)

In D2492#110907, @Stan wrote:

Stan` shoots himself in the foot: Shouldn't this use Status Effects? Could be done in V2.

I'm not sure how one could use Status Effects for this?

Add an autobuild status effect? Which deals construction damage

Stan updated this revision to Diff 11419.Feb 25 2020, 1:12 AM

Move the code to the Autobuildable component to decouple it as much as possible.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 165| 165| 	this.totalBuilderRate += buildRate;
| 166| 166| 
| 167| 167| 	return true;
| 168|    |-}
|    | 168|+};
| 169| 169| 
| 170| 170| /**
| 171| 171|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 200| 200| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 201| 201| 
| 202| 202| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 203|    |-}
|    | 203|+};
| 204| 204| 
| 205| 205| /**
| 206| 206|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 168| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 203| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1810/display/redirect

Stan marked 2 inline comments as done.Feb 25 2020, 1:23 AM
Stan updated this revision to Diff 11421.Feb 25 2020, 1:27 AM

Remove more noise and coupling.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 162| 162| 	this.totalBuilderRate += buildRate;
| 163| 163| 
| 164| 164| 	return true;
| 165|    |-}
|    | 165|+};
| 166| 166| 
| 167| 167| /**
| 168| 168|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 197| 197| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 198| 198| 
| 199| 199| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 200|    |-}
|    | 200|+};
| 201| 201| 
| 202| 202| /**
| 203| 203|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 165| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 200| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 370| »   »   var·pos·=·cmpPosition.GetPosition2D();
|    | [NORMAL] JSHintBear:
|    | 'pos' is already defined.

binaries/data/mods/public/simulation/components/Foundation.js
| 372| »   »   var·rot·=·cmpPosition.GetRotation();
|    | [NORMAL] JSHintBear:
|    | 'rot' is already defined.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1811/display/redirect

can you request review ?

Stan requested review of this revision.Feb 25 2020, 8:23 AM
Stan updated this revision to Diff 11422.

Remove message code.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 162| 162| 	this.totalBuilderRate += buildRate;
| 163| 163| 
| 164| 164| 	return true;
| 165|    |-}
|    | 165|+};
| 166| 166| 
| 167| 167| /**
| 168| 168|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 197| 197| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 198| 198| 
| 199| 199| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 200|    |-}
|    | 200|+};
| 201| 201| 
| 202| 202| /**
| 203| 203|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 165| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 200| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 370| »   »   var·pos·=·cmpPosition.GetPosition2D();
|    | [NORMAL] JSHintBear:
|    | 'pos' is already defined.

binaries/data/mods/public/simulation/components/Foundation.js
| 372| »   »   var·rot·=·cmpPosition.GetRotation();
|    | [NORMAL] JSHintBear:
|    | 'rot' is already defined.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1812/display/redirect

Silier added inline comments.Feb 25 2020, 11:56 AM
binaries/data/mods/public/simulation/components/AutoBuildable.js
71

this should now use this.rate dirrectly, same above

Freagarach added inline comments.Feb 25 2020, 7:34 PM
binaries/data/mods/public/simulation/components/AutoBuildable.js
40

If you use this.rate you can combine the two ifs :)

48

Could we possible deconstruct the AutoBuildable component when there is no cmpFoundation?

Stan updated this revision to Diff 11425.Feb 25 2020, 8:07 PM

Remove usage of this.GetRate()

Stan marked 3 inline comments as done.Feb 25 2020, 8:07 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/1294/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1813/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/390/display/redirect

Stan updated this revision to Diff 11427.Feb 25 2020, 8:12 PM

Fix missing this.

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 162| 162| 	this.totalBuilderRate += buildRate;
| 163| 163| 
| 164| 164| 	return true;
| 165|    |-}
|    | 165|+};
| 166| 166| 
| 167| 167| /**
| 168| 168|  * Adds a builder to the counter.
|    | [NORMAL] ESLintBear (semi):
|    | Missing semicolon.
|----|    | /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
|    |++++| /zpool0/trunk/binaries/data/mods/public/simulation/components/Foundation.js
| 197| 197| 		cmpVisual.SetVariable("numbuilders", this.builders.size);
| 198| 198| 
| 199| 199| 	Engine.PostMessage(this.entity, MT_FoundationBuildersChanged, { "to": this.GetBuilders() });
| 200|    |-}
|    | 200|+};
| 201| 201| 
| 202| 202| /**
| 203| 203|  * The build multiplier is a penalty that is applied to each builder.

binaries/data/mods/public/simulation/components/Foundation.js
| 165| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 200| }
|    | [NORMAL] JSHintBear:
|    | Missing semicolon.

binaries/data/mods/public/simulation/components/Foundation.js
| 370| »   »   var·pos·=·cmpPosition.GetPosition2D();
|    | [NORMAL] JSHintBear:
|    | 'pos' is already defined.

binaries/data/mods/public/simulation/components/Foundation.js
| 372| »   »   var·rot·=·cmpPosition.GetRotation();
|    | [NORMAL] JSHintBear:
|    | 'rot' is already defined.
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1814/display/redirect

Stan updated this revision to Diff 11430.Feb 25 2020, 9:39 PM

Remove template change.

Silier accepted this revision.Feb 25 2020, 9:39 PM
This revision is now accepted and ready to land.Feb 25 2020, 9:39 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1817/display/redirect

This revision was landed with ongoing or failed builds.Feb 25 2020, 10:30 PM
This revision was automatically updated to reflect the committed changes.