This patch corrects the descriptions and tooltips of siege technologies, consistently referring to the Siege class as “Siege Engines”, per https://trac.wildfiregames.com/wiki/EnglishStyleGuide#Units and standardizing the tooltip format to [class] [change] [attributes].
Details
Details
- Reviewers
Gallaecio - Commits
- rP22783: correct descriptions and tooltips of siege technologies
Check for mistakes.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 8929 Build 14655: Vulcan Build Jenkins Build 14654: arc lint + arc unit
Event Timeline
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/413/display/redirect
| binaries/data/mods/public/simulation/data/technologies/siege_bolt_accuracy.json | ||
|---|---|---|
| 13 | We might as well use "civ": instead of "notciv": here. | |
| binaries/data/mods/public/simulation/data/technologies/successors/special_parade_of_daphne.json | ||
|---|---|---|
| 3 | This implies that the units are automatically trained in large batches/battalions from now on? So by -> when? And remove battalions perhaps. | |
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/429/display/redirect