Page MenuHomeWildfire Games

can be brought → could be brought
ClosedPublic

Authored by Gallaecio on Mar 24 2019, 1:16 PM.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 7042
Build 11510: Vulcan BuildJenkins

Event Timeline

Gallaecio created this revision.Mar 24 2019, 1:16 PM
Vulcan added a subscriber: Vulcan.Mar 24 2019, 1:20 PM

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

Link to build: https://jenkins.wildfiregames.com/job/differential/1131/display/redirect

Stan accepted this revision.Mar 24 2019, 1:37 PM

Looks good to me.

This revision is now accepted and ready to land.Mar 24 2019, 1:37 PM

Line 103 appears to be identical to line 66, so you might want to correct that one too.
However, because neither the civ bonus nor the structure are implemented, it might be better to just purge both.

Stan added a comment.Mar 24 2019, 1:41 PM

Isn't that building the military coloney ?

"Must be constructed in neutral territory. Has no territory radius effect. Trains all 'mercenary' units."

The military colony does have a territory radius and a different name.

All civ files are full of not implemented, outdated, or incorrect information. Updating them would be a lot of work, but I think it's better than maintaining incorrect information.

This revision was automatically updated to reflect the committed changes.