Add support for multiple outputs

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 `:`.

Author: lafriks

pkg/commands/run.go

@@ -381,6 +381,8 @@ func (e *Executor) setExitCodeIfIssuesFound(issues []result.Issue) {
 	}
 }
 
+const defaultFileMode = 0644
+
 func (e *Executor) runAndPrint(ctx context.Context, args []string) error {
 	if err := e.goenv.Discover(ctx); err != nil {
 		e.log.Warnf("Failed to discover go env: %s", err)
@@ -400,44 +402,77 @@ func (e *Executor) runAndPrint(ctx context.Context, args []string) error {
 		return err // XXX: don't loose type
 	}
 
-	p, err := e.createPrinter()
-	if err != nil {
-		return err
+	formats := strings.Split(e.cfg.Output.Format, ",")
+	for _, format := range formats {
+		out := strings.SplitN(format, ":", 2)
+		if len(out) < 2 {
+			out = append(out, "")
+		}
+		w, shouldClose, err := e.createWriter(out[1])
+		if err != nil {
+			return fmt.Errorf("can't create output for %s: %w", out[1], err)
+		}
+
+		p, err := e.createPrinter(out[0], w)
+		if err != nil {
+			if file, ok := w.(io.Closer); shouldClose && ok {
+				file.Close()
+			}
+			return err
+		}
+		if err = p.Print(ctx, issues); err != nil {
+			if file, ok := w.(io.Closer); shouldClose && ok {
+				file.Close()
+			}
+			return fmt.Errorf("can't print %d issues: %s", len(issues), err)
+		}
+		if file, ok := w.(io.Closer); shouldClose && ok {
+			file.Close()
+		}
 	}
 
 	e.setExitCodeIfIssuesFound(issues)
 
-	if err = p.Print(ctx, issues); err != nil {
-		return fmt.Errorf("can't print %d issues: %s", len(issues), err)
-	}
-
 	e.fileCache.PrintStats(e.log)
 
 	return nil
 }
 
-func (e *Executor) createPrinter() (printers.Printer, error) {
+func (e *Executor) createWriter(path string) (io.Writer, bool, error) {
+	if path == "stdout" {
+		return logutils.StdOut, false, nil
+	}
+	if path == "stderr" {
+		return logutils.StdErr, false, nil
+	}
+	f, err := os.OpenFile(path, os.O_CREATE|os.O_TRUNC|os.O_WRONLY, defaultFileMode)
+	if err != nil {
+		return nil, false, err
+	}
+	return f, true, nil
+}
+
+func (e *Executor) createPrinter(format string, w io.Writer) (printers.Printer, error) {
 	var p printers.Printer
-	format := e.cfg.Output.Format
 	switch format {
 	case config.OutFormatJSON:
-		p = printers.NewJSON(&e.reportData)
+		p = printers.NewJSON(&e.reportData, w)
 	case config.OutFormatColoredLineNumber, config.OutFormatLineNumber:
 		p = printers.NewText(e.cfg.Output.PrintIssuedLine,
 			format == config.OutFormatColoredLineNumber, e.cfg.Output.PrintLinterName,
-			e.log.Child("text_printer"))
+			e.log.Child("text_printer"), w)
 	case config.OutFormatTab:
-		p = printers.NewTab(e.cfg.Output.PrintLinterName, e.log.Child("tab_printer"))
+		p = printers.NewTab(e.cfg.Output.PrintLinterName, e.log.Child("tab_printer"), w)
 	case config.OutFormatCheckstyle:
-		p = printers.NewCheckstyle()
+		p = printers.NewCheckstyle(w)
 	case config.OutFormatCodeClimate:
-		p = printers.NewCodeClimate()
+		p = printers.NewCodeClimate(w)
 	case config.OutFormatHTML:
-		p = printers.NewHTML()
+		p = printers.NewHTML(w)
 	case config.OutFormatJunitXML:
-		p = printers.NewJunitXML()
+		p = printers.NewJunitXML(w)
 	case config.OutFormatGithubActions:
-		p = printers.NewGithub()
+		p = printers.NewGithub(w)
 	default:
 		return nil, fmt.Errorf("unknown output format %s", format)
 	}

pkg/printers/checkstyle.go

@@ -4,10 +4,10 @@ import (
 	"context"
 	"encoding/xml"
 	"fmt"
+	"io"
 
 	"github.com/go-xmlfmt/xmlfmt"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
@@ -32,13 +32,17 @@ type checkstyleError struct {
 
 const defaultCheckstyleSeverity = "error"
 
-type Checkstyle struct{}
+type Checkstyle struct {
+	w io.Writer
+}
 
-func NewCheckstyle() *Checkstyle {
-	return &Checkstyle{}
+func NewCheckstyle(w io.Writer) *Checkstyle {
+	return &Checkstyle{
+		w: w,
+	}
 }
 
-func (Checkstyle) Print(ctx context.Context, issues []result.Issue) error {
+func (p Checkstyle) Print(ctx context.Context, issues []result.Issue) error {
 	out := checkstyleOutput{
 		Version: "5.0",
 	}
@@ -82,6 +86,6 @@ func (Checkstyle) Print(ctx context.Context, issues []result.Issue) error {
 		return err
 	}
 
-	fmt.Fprintf(logutils.StdOut, "%s%s\n", xml.Header, xmlfmt.FormatXML(string(data), "", "  "))
+	fmt.Fprintf(p.w, "%s%s\n", xml.Header, xmlfmt.FormatXML(string(data), "", "  "))
 	return nil
 }

pkg/printers/codeclimate.go

@@ -4,8 +4,8 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
+	"io"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
@@ -24,10 +24,13 @@ type CodeClimateIssue struct {
 }
 
 type CodeClimate struct {
+	w io.Writer
 }
 
-func NewCodeClimate() *CodeClimate {
-	return &CodeClimate{}
+func NewCodeClimate(w io.Writer) *CodeClimate {
+	return &CodeClimate{
+		w: w,
+	}
 }
 
 func (p CodeClimate) Print(ctx context.Context, issues []result.Issue) error {
@@ -52,6 +55,6 @@ func (p CodeClimate) Print(ctx context.Context, issues []result.Issue) error {
 		return err
 	}
 
-	fmt.Fprint(logutils.StdOut, string(outputJSON))
+	fmt.Fprint(p.w, string(outputJSON))
 	return nil
 }

pkg/printers/github.go

@@ -3,20 +3,23 @@ package printers
 import (
 	"context"
 	"fmt"
+	"io"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
 type github struct {
+	w io.Writer
 }
 
 const defaultGithubSeverity = "error"
 
 // NewGithub output format outputs issues according to GitHub actions format:
 // https://help.github.com/en/actions/reference/workflow-commands-for-github-actions#setting-an-error-message
-func NewGithub() Printer {
-	return &github{}
+func NewGithub(w io.Writer) Printer {
+	return &github{
+		w: w,
+	}
 }
 
 // print each line as: ::error file=app.js,line=10,col=15::Something went wrong
@@ -35,9 +38,9 @@ func formatIssueAsGithub(issue *result.Issue) string {
 	return ret
 }
 
-func (g *github) Print(_ context.Context, issues []result.Issue) error {
+func (p *github) Print(_ context.Context, issues []result.Issue) error {
 	for ind := range issues {
-		_, err := fmt.Fprintln(logutils.StdOut, formatIssueAsGithub(&issues[ind]))
+		_, err := fmt.Fprintln(p.w, formatIssueAsGithub(&issues[ind]))
 		if err != nil {
 			return err
 		}

pkg/printers/html.go

@@ -4,9 +4,9 @@ import (
 	"context"
 	"fmt"
 	"html/template"
+	"io"
 	"strings"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
@@ -123,13 +123,17 @@ type htmlIssue struct {
 	Code   string
 }
 
-type HTML struct{}
+type HTML struct {
+	w io.Writer
+}
 
-func NewHTML() *HTML {
-	return &HTML{}
+func NewHTML(w io.Writer) *HTML {
+	return &HTML{
+		w: w,
+	}
 }
 
-func (h HTML) Print(_ context.Context, issues []result.Issue) error {
+func (p HTML) Print(_ context.Context, issues []result.Issue) error {
 	var htmlIssues []htmlIssue
 
 	for i := range issues {
@@ -151,5 +155,5 @@ func (h HTML) Print(_ context.Context, issues []result.Issue) error {
 		return err
 	}
 
-	return t.Execute(logutils.StdOut, struct{ Issues []htmlIssue }{Issues: htmlIssues})
+	return t.Execute(p.w, struct{ Issues []htmlIssue }{Issues: htmlIssues})
 }

pkg/printers/json.go

@@ -4,19 +4,21 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
+	"io"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/report"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
 type JSON struct {
 	rd *report.Data
+	w  io.Writer
 }
 
-func NewJSON(rd *report.Data) *JSON {
+func NewJSON(rd *report.Data, w io.Writer) *JSON {
 	return &JSON{
 		rd: rd,
+		w:  w,
 	}
 }
 
@@ -39,6 +41,6 @@ func (p JSON) Print(ctx context.Context, issues []result.Issue) error {
 		return err
 	}
 
-	fmt.Fprint(logutils.StdOut, string(outputJSON))
+	fmt.Fprint(p.w, string(outputJSON))
 	return nil
 }

pkg/printers/junitxml.go

@@ -3,9 +3,9 @@ package printers
 import (
 	"context"
 	"encoding/xml"
+	"io"
 	"strings"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
@@ -35,13 +35,16 @@ type failureXML struct {
 }
 
 type JunitXML struct {
+	w io.Writer
 }
 
-func NewJunitXML() *JunitXML {
-	return &JunitXML{}
+func NewJunitXML(w io.Writer) *JunitXML {
+	return &JunitXML{
+		w: w,
+	}
 }
 
-func (JunitXML) Print(ctx context.Context, issues []result.Issue) error {
+func (p JunitXML) Print(ctx context.Context, issues []result.Issue) error {
 	suites := make(map[string]testSuiteXML) // use a map to group by file
 
 	for ind := range issues {
@@ -70,7 +73,7 @@ func (JunitXML) Print(ctx context.Context, issues []result.Issue) error {
 		res.TestSuites = append(res.TestSuites, val)
 	}
 
-	enc := xml.NewEncoder(logutils.StdOut)
+	enc := xml.NewEncoder(p.w)
 	enc.Indent("", "  ")
 	if err := enc.Encode(res); err != nil {
 		return err

pkg/printers/tab.go

@@ -15,12 +15,14 @@ import (
 type Tab struct {
 	printLinterName bool
 	log             logutils.Log
+	w               io.Writer
 }
 
-func NewTab(printLinterName bool, log logutils.Log) *Tab {
+func NewTab(printLinterName bool, log logutils.Log, w io.Writer) *Tab {
 	return &Tab{
 		printLinterName: printLinterName,
 		log:             log,
+		w:               w,
 	}
 }
 
@@ -30,7 +32,7 @@ func (p Tab) SprintfColored(ca color.Attribute, format string, args ...interface
 }
 
 func (p *Tab) Print(ctx context.Context, issues []result.Issue) error {
-	w := tabwriter.NewWriter(logutils.StdOut, 0, 0, 2, ' ', 0)
+	w := tabwriter.NewWriter(p.w, 0, 0, 2, ' ', 0)
 
 	for i := range issues {
 		p.printIssue(&issues[i], w)