Skip to content

Commit

Permalink
fixed most init errors in channels/store/storetest dir | 24 files edi…
Browse files Browse the repository at this point in the history
…ted (mattermost#26522)

Co-authored-by: Ben Schumacher <ben.schumacher@mattermost.com>
Co-authored-by: Agniva De Sarker <agnivade@yahoo.co.in>
  • Loading branch information
3 people authored Oct 29, 2024
1 parent 5dba6e0 commit 71bf777
Show file tree
Hide file tree
Showing 48 changed files with 3,018 additions and 3,020 deletions.
4 changes: 1 addition & 3 deletions server/.golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@ linters:
- errcheck

issues:
exclude-dirs:
- channels/store/storetest/mocks
exclude-rules:
- linters:
# ignore unused warnings from enterprise code
Expand All @@ -39,8 +37,8 @@ issues:

- linters:
- revive
text: "var-naming|error-naming|exported|increment-decrement|error-strings|if-return|unused-parameter|blank-imports|empty-block"
# We need to fix the unused parameter issues and remove the exception.
text: "var-naming|error-naming|exported|increment-decrement|error-strings|if-return|unused-parameter|blank-imports|empty-block"

# Use a seperate set of exception for the enterprise code to allow community members to fix only the open source issues.
- linters:
Expand Down
2 changes: 1 addition & 1 deletion server/channels/app/channel.go
Original file line number Diff line number Diff line change
Expand Up @@ -2954,7 +2954,7 @@ func (a *App) SearchAllChannels(c request.CTX, term string, opts model.ChannelSe
ExcludeGroupConstrained: opts.ExcludeGroupConstrained,
PolicyID: opts.PolicyID,
IncludePolicyID: opts.IncludePolicyID,
IncludeSearchById: opts.IncludeSearchById,
IncludeSearchByID: opts.IncludeSearchById,
ExcludeRemote: opts.ExcludeRemote,
ExcludePolicyConstrained: opts.ExcludePolicyConstrained,
Public: opts.Public,
Expand Down
2 changes: 1 addition & 1 deletion server/channels/app/post.go
Original file line number Diff line number Diff line change
Expand Up @@ -2151,7 +2151,7 @@ func (a *App) SetPostReminder(rctx request.CTX, postID, userID string, targetTim
CreateAt: model.GetMillis(),
UserId: userID,
RootId: postID,
ChannelId: metadata.ChannelId,
ChannelId: metadata.ChannelID,
// It's okay to keep this non-translated. This is just a fallback.
// The webapp will parse the timestamp and show that in user's local timezone.
Message: fmt.Sprintf("You will be reminded about %s by @%s at %s", permalink, metadata.Username, parsedTime),
Expand Down
240 changes: 120 additions & 120 deletions server/channels/store/opentracinglayer/opentracinglayer.go

Large diffs are not rendered by default.

Loading

0 comments on commit 71bf777

Please sign in to comment.