Add support for multiple outputs

[lafriks]

Outputs can be specified using --out-format by separating them by comma.

Support providing file path by separating format name and path by colon symbol :.

Fixes #481

[boring-cyborg]

Hey, thank you for opening your first Pull Request !

[CLAassistant]

All committers have signed the CLA.

[SVilgelm]

Thank you very much, it really logs good, but definitely requires some additional tests to cover the files and stderr|stdout as well

[lafriks]

stdout should be already covered as default but can be easily added to have for one format to be specified.

I can easily add file output(s) test case.

For stderr I would need some guidance as I'm not very familiar with testing helpers used in this project.

[lafriks]

@SVilgelm I added test cases for stderr and file outputs. As testing helper command stderr are combined with stdout it's not clean way to test this. Only if stdout and stderr would be read into separate variables and that would break quite some other tests as they expect combined output.

[SVilgelm]

LGTM

[BradErz]

Is there any ETA on this one being merged?

[ldez]

@lafriks the PRs from a GitHub organization cannot be edited by the maintainers, I recommend not using this pattern to create PR for OSS.

So I am not able to rebase the PR, I'm also not able to add commits.

Can you recreate your PR from your/a personal account?
(tips: you can transfer the repo from your fork to your personal account to avoid recreating a PR)

If you can't, the review will be complex.

[lafriks]

@ldez I have added you write permissions to forked repository. I use such approach to somehow separate my own developed projects and OSS projects I contribute to only time to time 😄

[ldez]

I know very well this pattern, but still, I recommend not using it.
If you want to separate fork and sources, it's better to create a user instead of an organization.

Note: I don't understand why do you want to separate forks and sources.

[lafriks]

Off-topic: I just started to do that some time ago because I kept having too many repositories that can't really be deleted because of open PR and it can take quite some time to get merged, sometimes months

[ldez]

LGTM

[golangci-automator]

Hey, @lafriks — we just merged your PR to golangci-lint! 🔥🚀

golangci-lint is built by awesome people like you. Let us say “thanks”: we just invited you to join the GolangCI organization on GitHub.
This will add you to our team of maintainers. Accept the invite by visiting this link.

By joining the team, you’ll be able to label issues, review pull requests, and merge approved pull requests.
More information about contributing is here.

Thanks again!

[fho]

@lafriks thanks a lot for implementing it :-)