FAForever / fa

Lua code for FAF
226 stars 232 forks source link

Remove the 'CommandCaps' field from various blueprints that do not need it #6158

Closed Basilisk3 closed 4 months ago

Basilisk3 commented 5 months ago

Description of the proposed changes

When these fields are set to false, they can be safely removed from the blueprint.

Testing done on the proposed changes

All units I tested still functioned normally.

Checklist

Basilisk3 commented 5 months ago

@Garanas I will remove the rest in another PR once this is merged. This one has become quite large already.

Garanas commented 5 months ago

Do you use a tool to make these changes?

Basilisk3 commented 5 months ago

I search the unit folder for all instances I want to remove, and then replace them with nothing via the replace feature.

Garanas commented 5 months ago

Fair enough 👍

Basilisk3 commented 4 months ago

CommandCaps are part of the .bp index of both unit-databases, but they do not show up in any calculations. Correct me if I am wrong, but this should not cause any issues.