Refactoring idea: Run method
Ouverte
enhancement
help wanted
- Langage dominant
- C
- Étoiles
- 178
- Forks
- 12
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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 https://github.com/alegrey91/harpoon/pull/60#discussion_r1894676098_
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Évaluation
Cette issue n'a pas encore été évaluée.