help wantedtechnical debt
Metriche repository
- Star
- (42 stelle)
- Metriche merge PR
- (Nessuna PR mergiata in 30 g)
Descrizione
Currently classes are used more or less as libraries rather than in a true OO fashion, e.g.
- State is passed everywhere via parameters (e.g.
dictandg), rather than being stored in instances. - The instances have little or no state, and yet the methods are mostly instance methods not class methods.
- There is no relationship modelled between classes, whereas there is ample opportunity for composition via dependency injection.
- Core objects such as playbooks, roles, tasks, templates, variables etc. are modelled as
Hashes rather than as objects. Commonality between the objects (e.g. name, fqn, parent, originating file path) is not enforced in any rigorous way - this is crying out for inheritance.