Skip to content

Refactoring idea: Run method #63

@ccoVeille

Description

@ccoVeille

Run method use channels for errors and outputs, but also use a fmt.Fprintf(os.Stderr

We could check what could be done to simplify things. Ideas such as:

  • check of the method signature shouldn't be updated to return an error. Handling error with printing is strange to me,it should be moved to higher level.
  • check if other errors currently send to outputCh or errCh shouldn't also be refactored

Originally posted by @ccoVeille in #60 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requesthelp wantedExtra attention is needed

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions