Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

chore: limit direct uses of os.Stdout/os.Stderr/os.Stdin #278

Merged
merged 1 commit into from
Mar 10, 2021

Conversation

cmoog
Copy link
Contributor

@cmoog cmoog commented Mar 10, 2021

By moving the out/err/in channels to the cobra config,
it will be easier to add fast and isolated unit tests.

By moving the out/err/in channels to the cobra config,
it will be easier to add fast and isolated unit tests.
@cmoog cmoog requested a review from jawnsy March 10, 2021 19:40
Copy link
Contributor

@jawnsy jawnsy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet, LGTM 👍

@cmoog cmoog merged commit bc391d9 into master Mar 10, 2021
@cmoog cmoog deleted the cmoog/input-output branch March 10, 2021 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants