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

All IOutput implementations are always used #10

Closed
mgroves opened this issue Apr 26, 2016 · 2 comments
Closed

All IOutput implementations are always used #10

mgroves opened this issue Apr 26, 2016 · 2 comments

Comments

@mgroves
Copy link
Contributor

mgroves commented Apr 26, 2016

Unless I'm misunderstanding something, it seems like all IOutput implementations are always used for each 'error' response.

One thing that might be nice is that currently your program will try to output to ALL implementation of IOutput. It might be better to all that to be configurable (e.g. in app config).

Something like:

So, if you keep adding output options (as in issue #6), you don't have to keep writing to all of them. Does that sound like a useful feature?

@hmol
Copy link
Owner

hmol commented Apr 26, 2016

In App.config I have a setting OnlyReportBrokenLinksToOutput, if true then only error-links will be outputted, if false all links will be outputted.
Outputting to several different "outputs" at the same time is confusing and not necessary. So it sounds very useful to make this configurable :bowtie:.

@mgroves
Copy link
Contributor Author

mgroves commented Apr 30, 2016

Looks like @wizhi beat me to it!

I'll wait for this to get merged in before I finish #6

@hmol hmol closed this as completed in f0c34d5 May 3, 2016
hmol added a commit that referenced this issue May 3, 2016
Make Output providers configurable, fixes #10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants