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

5x faster for empty strings, not 30x according to previous me https://code.wildfiregames.com/D2168#90278

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

the “” technology

Units have different gather rates for resource subtypes, therefore it makes sense to mention both the subtype (what you gather) and the resource (what you gather) in the tooltip. We already have “Harvest grain for food.” in the field tooltip and my suggestion is to follow that format for the other resource entity tooltips. Flora has wood.tree or food.fruit. And animals have to be killed before resources can be collected, hence the “Kill to”.
But yeah, sentences should be proper English; robot language is undesirable.
(Also, lumber is something quite different than wood; look it up in a dictionary.)

I have no strong opinion on the tag changes; if someone else agrees with them, let’s go ahead. Otherwise, let’s keep them as before to minimize string changes.

In mark-up languages it's customary to close tags on the same line, e.g. LaTeX \textbf{lorem}, HTML <b>ipsum</b>, MarkDown **dolor**, hence my preference for [font="sans-bold-14"]sit[font="sans-14"] in this text file, which also makes it easier to see if a tag is forgotten.

I pinged them on twitter. We'll see.

Thank you for the review @JoshuaJB !

In rP22673#36250, @elexis wrote:

It might also be that the servers are timing out, not client(you). Unless you have someone already taking care of it, you might want to try to commit smaller batches of modified files if that is possible.
Thanks for taking care of this issue Alexandermb!

Was playing and got these errors:
ERROR: Could not load mesh 'art/meshes/skeletal/new/armor/m_tunic_short.dae'
ERROR: CObjectEntry::BuildVariation(): Model art/meshes/skeletal/new/armor/m_tunic_short.dae failed to load
ERROR: Failed to build prop model "units/seleucids/elephantry_c_r.xml" on actor "turret_sele_champ_ele"
ERROR: Could not load mesh 'art/meshes/skeletal/new/armor/m_tunic_short.dae'
ERROR: CObjectEntry::BuildVariation(): Model art/meshes/skeletal/new/armor/m_tunic_short.dae failed to load
ERROR: Failed to build prop model "units/seleucids/elephantry_c_r.xml" on actor "turret_sele_champ_ele"

In D1808#90733, @Nescio wrote:

Before I can update this, we still need to find a proper format for the formation and garrison aura types.

Maybe what we need first is to have two completely different words to refer to (1) formation as in a way a group of units can organize themselves and (2) formation as in a specific group of units. For example, if we used ‘group’ for the latter, “Same group [class] [change] [attributes]” may work.

A group is an arbitrary selection of entities, which can be defined by e.g. Ctrl+1 and selected by pressing 1 or clicking on the group panel at the left of the screen.

In D2006#90693, @Nescio wrote:

So basically you prefer [verb] for [resource] instead of [verb] [subtype] for [resource]?

I meant what @Freagarach said, and as I suspected is a no-go.

Passing callback functions as arguments instead of names fixed in rP22676.