Refactor transform-classes #7202
Labels
i: discussion
outdated
A closed issue/PR that is archived due to age. Recommended to make a new issue
PR: Polish 💅
A type of pull request used for our changelog categories
At the moment transform-classes has 2 modes - regular one and loose. The loose mode is extending VanillaTransformer class. It seems to me that this hierarchical structure here is clunky and harder to maintain.
Don't know how others feel, but imho it would be nice to refactor this into a flat structure and reuse code between 2 transforms if possible, but more in a composition manner than with inheritance.
I'd like to have at least 2-3 upvotes from other core team members for this, before anyone starts to work on this.
The text was updated successfully, but these errors were encountered: