After deleting all the old code in T388227: De-facto remove code for Special:EditGrowthConfig, AbstractDataConfigurationLoader only has one subclass left. That probably does not make sense and should be merged back.
Also, we should check for other abstract classes or PHP Traits like the above that are maybe no longer needed.
- AbstractDataConfigurationLoader <-> CommunityConfigurationLoader
- AbstractStructuredMentorProvider <-> CommunityStructuredMentorProvider
- AbstractStructuredMentorWriter <-> CommunityStructuredMentorWriter
- Think about ConfigurationValidator - is all of this still doing something meaningful, or is it just duplicating validation work that is actually handled by CommunityConfiguration and the respective Schema now? Should we keep it as it is?