Skip to content

Refactor transform-classes #7202

@Andarist

Description

@Andarist

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    PR: Polish 💅A type of pull request used for our changelog categoriesi: discussionoutdatedA closed issue/PR that is archived due to age. Recommended to make a new issue

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions