Add `!` postfix to indicate call has side-effect

Just a small cosmetic follow up to !18005 (merged)

Since we're mutating the argument passed into remove_feature_dependent_sub_relations, I had suggested to bang-ify it to indicate we're performing a side-effect.

Does this MR meet the acceptance criteria?

Conformity

• Code review guidelines
• Merge request performance guidelines
• Style guides