Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

q: App.Writer is being set twice #1096

Closed
Nokel81 opened this issue Mar 25, 2020 · 2 comments · Fixed by #1100
Closed

q: App.Writer is being set twice #1096

Nokel81 opened this issue Mar 25, 2020 · 2 comments · Fixed by #1100
Labels
area/v2 relates to / is being considered for v2 kind/question someone asking a question status/triage maintainers still need to look into this

Comments

@Nokel81
Copy link

Nokel81 commented Mar 25, 2020

I was wondering why there is two instance of setting *App.Writer to os.Stdout in *App.Setup()?

And *App.ErrWriter is not being set at all. Is that because it is going to be deprecated?

@Nokel81 Nokel81 added area/v2 relates to / is being considered for v2 kind/question someone asking a question status/triage maintainers still need to look into this labels Mar 25, 2020
@coilysiren
Copy link
Member

I was wondering why there is two instance of setting *App.Writer to os.Stdout in *App.Setup()?

🤷‍♀

And *App.ErrWriter is not being set at all. Is that because it is going to be deprecated?

🤷‍♀

@coilysiren
Copy link
Member

@Nokel81 if you would like to change either of those behaviors, I would happily review PRs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/v2 relates to / is being considered for v2 kind/question someone asking a question status/triage maintainers still need to look into this
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants