Another "rule of thumb" for writing maintainable code is "Don't Repeat Yourself" (DRY). It means that, when possible, you should avoid writing the same code in multiple places. Repeating code can be bad because:
Your manager noticed that "Age of Dragons" has a lot of repetitive code. She's asked you to update the fight_soldiers function so that the DPS (damage-per-second) calculation is only written once.
Notice how these two lines are practically identical:
soldier_one_dps = soldier_one["damage"] * soldier_one["attacks_per_second"]
soldier_two_dps = soldier_two["damage"] * soldier_two["attacks_per_second"]
The soldier with the greater DPS will win the fight!