Page MenuHomeWildfire Games

-ConquestCritical clear-up
Needs ReviewPublic

Authored by Nescio on Sep 13 2018, 5:23 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Currently the "ConquestCritical" class is added to the basic template_structure.xml file and subsequently removed in the majority of cases. This is quite ugly. A much neater way is to add it only to the cases that actually have it (Civic, Military, Wonder, and several special structures), but not to the basic parent template.

Test Plan

Effectively nothing ought to be changed.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6365
Build 10552: Vulcan BuildJenkins
Build 10551: arc lint + arc unit

Event Timeline

Nescio created this revision.Sep 13 2018, 5:23 PM
Vulcan added a subscriber: Vulcan.Sep 13 2018, 5:29 PM

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

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