laverdet / xxscreeps

Other
71 stars 16 forks source link

fixed docker file and rampart construction sites being un-walkable #55

Closed n-kri closed 2 years ago

n-kri commented 2 years ago

closes #50

n-kri commented 2 years ago

Test output:

15 1.514 Store.........

15 1.515 Common: Open store....

15 1.516 Common: Restricted store....

15 1.516 Common: Single store....

15 1.516 Movement..........

15 1.517 Room.

15 1.517 Pull........

15 1.517 ramparts..

15 1.517 Observer....

15 1.517 Roads..

15 1.517 Spawn......

laverdet commented 2 years ago

Thanks for the PR. a08f826dfe90e75864af8e9440b816245d58c86e fixes the travis build, so I'll put the docker workaround out of this patch and just keep the obstacle change. It probably makes sense to update ConstructionTraits as well, since this was the only place that makes use of the undefined type.

obstacle actually used to be false but it was changed to undefined here 2a0fd3992f10459a67c717b1cf3d763f21f50aa5 though I didn't document specifically which issue I was fixing at the time. The tests will be a big help here.

laverdet commented 2 years ago

Thanks, this was merged in f92550cbfc5b2064f90d2effbbe6c82623f05bcf