https://github.com/wesnoth/wesnoth/blob/f33330de19a24a256f028c4d259f19d363f19a25/src/help/help_impl.cpp#L277-L284

The current warning message should only be displayed if parts[0] is not recognized. If parts[1] is missing it should mention a [race] or an [era] with no id key respectively.

(Alternatively this whole internal config generation should be done away with. It seems like ther

Refactor classes (including tests) to resolve dependencies out of the IoC container with $this->dep = app($class::class) in the constructor, instead of through the constructor method arguments.

Constructor arg lists get quite long (see PopulationCalculator), and needs fixing test constructors whenever this changes.

Beep, boop, I'm a bot! Th

Something like Gold/Silver/Bronze medal icons would be neat