diff --git a/bridge/matrix/appservice.go b/bridge/matrix/appservice.go index 25cabeba..a2e3d84a 100644 --- a/bridge/matrix/appservice.go +++ b/bridge/matrix/appservice.go @@ -120,7 +120,7 @@ func (a *AppServiceNamespaces) containsRoom(roomID id.RoomID) bool { return false } -// nolint: wrapcheck +//nolint:wrapcheck func (b *Bmatrix) startAppService() error { wrapper := b.appService // TODO: detect service completion and rerun automatically diff --git a/bridge/matrix/handlers.go b/bridge/matrix/handlers.go index bea81884..f6c91fed 100644 --- a/bridge/matrix/handlers.go +++ b/bridge/matrix/handlers.go @@ -38,7 +38,7 @@ func (b *Bmatrix) ignoreBridgingEvents(ev *event.Event) bool { return false } -//nolint: funlen +//nolint:funlen func (b *Bmatrix) handleEvent(origin EventOrigin, ev *event.Event) { b.RLock() channel, ok := b.RoomMap[ev.RoomID] @@ -170,7 +170,7 @@ func (b *Bmatrix) handleMessage(rmsg config.Message, ev *event.Event) { rmsg.Avatar = b.getAvatarURL(ev.RoomID, ev.Sender) - //nolint: exhaustive + //nolint:exhaustive switch msg.MsgType { case event.MsgEmote: // Do we have a /me action @@ -264,7 +264,7 @@ func (b *Bmatrix) handleUploadFiles(msg *config.Message, channel id.RoomID) (str } // handleUploadFile handles native upload of a file. -//nolint: funlen +//nolint:funlen func (b *Bmatrix) handleUploadFile(msg *config.Message, channel id.RoomID, fi *config.FileInfo) { content := bytes.NewReader(*fi.Data) sp := strings.Split(fi.Name, ".") diff --git a/bridge/matrix/helpers.go b/bridge/matrix/helpers.go index b4580831..7a6f566c 100644 --- a/bridge/matrix/helpers.go +++ b/bridge/matrix/helpers.go @@ -247,7 +247,7 @@ type SendMessageEventWrapper struct { inner *matrix.Client } -//nolint: wrapcheck +//nolint:wrapcheck func (w SendMessageEventWrapper) SendMessageEvent(roomID id.RoomID, eventType event.Type, contentJSON interface{}) (*matrix.RespSendEvent, error) { return w.inner.SendMessageEvent(roomID, eventType, contentJSON) } @@ -286,7 +286,7 @@ func (b *Bmatrix) uploadAvatar(channelID id.RoomID, mxid id.UserID, intent *matr return nil } - //nolint: exhaustruct + //nolint:exhaustruct matrixResp, err := b.mc.UploadMedia(matrix.ReqUploadMedia{ ContentBytes: data, ContentType: resp.Header.Get("Content-Type"), @@ -305,7 +305,7 @@ func (b *Bmatrix) uploadAvatar(channelID id.RoomID, mxid id.UserID, intent *matr return nil } -//nolint: wrapcheck +//nolint:wrapcheck func (b *Bmatrix) sendMessageEventWithRetries(channel id.RoomID, message event.MessageEventContent, username string, avatarURL string) (string, error) { var ( resp *matrix.RespSendEvent @@ -333,7 +333,7 @@ func (b *Bmatrix) sendMessageEventWithRetries(channel id.RoomID, message event.M // if we can't change the display name it's not great but not the end of the world either, ignore it // TODO: do not perform this action on every message, with an in-memory cache or something _ = intent.SetDisplayName(username) - //nolint: errcheck + //nolint:errcheck go b.uploadAvatar(channel, id.UserID(bridgeUserID), intent, avatarURL) client = intent } else { diff --git a/bridge/matrix/matrix.go b/bridge/matrix/matrix.go index ae76819a..b57813c4 100644 --- a/bridge/matrix/matrix.go +++ b/bridge/matrix/matrix.go @@ -1,4 +1,4 @@ -// nolint: exhaustivestruct +//nolint:exhaustivestruct package bmatrix import ( @@ -80,7 +80,7 @@ func (b *Bmatrix) Connect() error { resp, err := b.mc.Login(&matrix.ReqLogin{ Type: matrix.AuthTypePassword, Password: b.GetString("Password"), - Identifier: matrix.UserIdentifier{Type: matrix.IdentifierTypeUser, User: b.GetString("Login")}, //nolint: exhaustruct + Identifier: matrix.UserIdentifier{Type: matrix.IdentifierTypeUser, User: b.GetString("Login")}, //nolint:exhaustruct StoreCredentials: true, }) if err != nil { @@ -195,7 +195,7 @@ func (b *Bmatrix) Start() error { return nil } -//nolint: funlen,gocognit,gocyclo +//nolint:funlen,gocognit,gocyclo func (b *Bmatrix) Send(msg config.Message) (string, error) { b.Log.Debugf("=> Sending %#v", msg) @@ -334,7 +334,8 @@ func (b *Bmatrix) Send(msg config.Message) (string, error) { // DontProcessOldEvents returns true if a sync event should be considered for further processing. // We use that function to filter out events we have already read. -//nolint: gocognit +// +//nolint:gocognit func (b *Bmatrix) DontProcessOldEvents(resp *matrix.RespSync, since string) bool { // we only filter old events in the initial sync(), because subsequent sync() // (where since != "") should only return new events