Skip to content

Refactor modifier classes and add docstrings #1641

Open
@matthewfeickert

Description

@matthewfeickert

PR #1625 added the modifier builder and combine classes, but they might be good candidates for a refactor that allows for a base class.

They also all need docstrings. This needs to happen before v0.7.0 so that we don't have an undocumented API.

Originally posted by @matthewfeickert in #1625 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    APIChanges the public APIdocsDocumentation relatedquestionFurther information is requestedrefactorA code change that neither fixes a bug nor adds a feature

    Type

    No type

    Projects

    Status

    To do

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions