$ golangci-lint run
log/log.go:189:8: QF1004: could use strings.ReplaceAll instead (staticcheck)
        msg = strings.Replace(msg, "\n", "\\n", -1)
              ^
log/log.go:235:9: QF1004: could use strings.ReplaceAll instead (staticcheck)
                msg = strings.Replace(msg, "\n", "\\n", -1)
                      ^
mocks/sa.go:249:2: QF1003: could use tagged switch on req.Id (staticcheck)
        if req.Id == 2 {
        ^
policy/pa.go:391:16: QF1004: could use strings.Split instead (staticcheck)
        splitEmail := strings.SplitN(email.Address, "@", -1)
                      ^
publisher/publisher_test.go:70:3: QF1007: could merge conditional assignment into variable declaration (staticcheck)
                precert := false
                ^
publisher/publisher_test.go:95:3: QF1007: could merge conditional assignment into variable declaration (staticcheck)
                precert := false
                ^
ra/ra_test.go:670:18: QF1009: probably want to use time.Time.Equal instead (staticcheck)
                test.Assert(t, *challenge.Validated == expectedValidated, "Validated timestamp incorrect or missing")
                               ^
ra/ra_test.go:882:17: QF1009: probably want to use time.Time.Equal instead (staticcheck)
        test.Assert(t, *challenge.Validated == expectedValidated, "Validated timestamp incorrect or missing")
                       ^
test/certs/webpki.go:110:16: QF1004: could use strings.ReplaceAll instead (staticcheck)
                        fileName := strings.Replace(name, " ", "-", -1)
                                    ^
test/s3-test-srv/main.go:25:2: QF1003: could use tagged switch on r.Method (staticcheck)
        if r.Method == "PUT" {
        ^
10 issues:
* staticcheck: 10