chore(deps): bump github.com/bwmarrin/discordgo from 0.27.1 to 0.28.1 (#45)

dependabot[bot] · rancoud · web-flow · commit ca1d94b6d07a · 2024-04-27T02:37:11.000+02:00
* chore(deps): bump github.com/bwmarrin/discordgo from 0.27.1 to 0.28.1 Bumps [github.com/bwmarrin/discordgo](https://github.com/bwmarrin/discordgo) from 0.27.1 to 0.28.1. - [Release notes](https://github.com/bwmarrin/discordgo/releases) - [Commits](bwmarrin/discordgo@v0.27.1...v0.28.1) --- updated-dependencies: - dependency-name: github.com/bwmarrin/discordgo dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * fix golangci-lint --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Sébastien Rancoud <rancoud@users.noreply.github.com>
diff --git a/.golangci.yml b/.golangci.yml
@@ -15,6 +15,11 @@ linters:
     - scopelint
     - structcheck
     - varcheck
+    - intrange
+
+issues:
+  max-issues-per-linter: 0
+  max-same-issues: 0
 
 linters-settings:
   lll:
diff --git a/configuration/configuration.go b/configuration/configuration.go
@@ -56,6 +56,7 @@ func ReadConfiguration(fsys fs.FS, filename string) (*Configuration, error) {
 
 	config := Configuration{}
 
+	//nolint:musttag
 	err = json.Unmarshal(filedata, &config)
 	if err != nil {
 		return nil, fmt.Errorf("%w", err)
diff --git a/configuration/configuration_test.go b/configuration/configuration_test.go
@@ -172,8 +172,6 @@ func TestReadConfiguration_Errors(t *testing.T) {
 	}
 
 	for testCaseName, testCase := range testCases {
-		testCaseName, testCase := testCaseName, testCase
-
 		t.Run(testCaseName, func(tt *testing.T) {
 			tt.Parallel()
 
diff --git a/go.mod b/go.mod
@@ -3,7 +3,7 @@ module github.com/blueprintue/discord-bot
 go 1.22
 
 require (
-	github.com/bwmarrin/discordgo v0.27.1
+	github.com/bwmarrin/discordgo v0.28.1
 	github.com/crazy-max/gohealthchecks v0.4.1
 	github.com/ilya1st/rotatewriter v0.0.0-20171126183947-3df0c1a3ed6d
 	github.com/rs/zerolog v1.32.0
diff --git a/go.sum b/go.sum
@@ -1,5 +1,5 @@
-github.com/bwmarrin/discordgo v0.27.1 h1:ib9AIc/dom1E/fSIulrBwnez0CToJE113ZGt4HoliGY=
-github.com/bwmarrin/discordgo v0.27.1/go.mod h1:NJZpH+1AfhIcyQsPeuBKsUtYrRnjkyu0kIVMCHkZtRY=
+github.com/bwmarrin/discordgo v0.28.1 h1:gXsuo2GBO7NbR6uqmrrBDplPUx2T3nzu775q/Rd1aG4=
+github.com/bwmarrin/discordgo v0.28.1/go.mod h1:NJZpH+1AfhIcyQsPeuBKsUtYrRnjkyu0kIVMCHkZtRY=
 github.com/coreos/go-systemd/v22 v22.5.0/go.mod h1:Y58oyj3AT4RCenI/lSvhwexgC+NSVTIJ3seZv2GcEnc=
 github.com/crazy-max/gohealthchecks v0.4.1 h1:gbjZzF/GxwDyP78u37B2/c2iQfq8BEjAHS3eBLM6FcQ=
 github.com/crazy-max/gohealthchecks v0.4.1/go.mod h1:gkT8QSdEXZJahyswdTGDbd+q20fWm0DmWW7TWBNtgJg=
diff --git a/hack/lint.Dockerfile b/hack/lint.Dockerfile
@@ -5,9 +5,9 @@ ENV GOFLAGS="-buildvcs=false"
 RUN apk add --no-cache gcc linux-headers musl-dev
 WORKDIR /src
 
-FROM golangci/golangci-lint:latest-alpine AS golangci-lint
+FROM golangci/golangci-lint:v1.57.2-alpine AS golangci-lint
 FROM base AS lint
 RUN --mount=type=bind,target=. \
-    --mount=type=cache,target=/root/.cache \
-    --mount=from=golangci-lint,source=/usr/bin/golangci-lint,target=/usr/bin/golangci-lint \
+  --mount=type=cache,target=/root/.cache \
+  --mount=from=golangci-lint,source=/usr/bin/golangci-lint,target=/usr/bin/golangci-lint \
   golangci-lint run ./...
diff --git a/healthchecks/healthchecks_test.go b/healthchecks/healthchecks_test.go
@@ -89,8 +89,6 @@ func TestNewHealthchecksManager_ErrorHasValidConfigurationInFile(t *testing.T) {
 	}
 
 	for testCaseName, testCase := range testCases {
-		testCaseName, testCase := testCaseName, testCase
-
 		t.Run(testCaseName, func(tt *testing.T) {
 			var bufferLogs bytes.Buffer
 			log.Logger = zerolog.New(&bufferLogs).Level(zerolog.TraceLevel).With().Logger()
diff --git a/welcome/welcome_handler_test.go b/welcome/welcome_handler_test.go
@@ -87,7 +87,7 @@ func TestHandlers_OnMessageReactionAdd(t *testing.T) {
 		[]requestTest{
 			{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 			{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/guilds/guild-123/members/user-id-456/roles/role-123"},
 		},
@@ -277,7 +277,7 @@ func TestHandlers_OnMessageReactionAdd_Errors(t *testing.T) {
 		[]requestTest{
 			{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 			{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/guilds/guild-123/members/user-id-456/roles/role-123"},
 		},
@@ -377,7 +377,7 @@ func TestHandlers_OnMessageReactionRemove(t *testing.T) {
 		[]requestTest{
 			{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 			{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			{method: "DELETE", host: "discord.com", uri: "/api/v9/guilds/guild-123/members/user-id-789/roles/role-123"},
 		},
@@ -566,7 +566,7 @@ func TestHandlers_OnMessageReactionRemove_Errors(t *testing.T) {
 		[]requestTest{
 			{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 			{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+				body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 			{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			{method: "DELETE", host: "discord.com", uri: "/api/v9/guilds/guild-123/members/user-id-789/roles/role-123"},
 		},
diff --git a/welcome/welcome_run_test.go b/welcome/welcome_run_test.go
@@ -89,7 +89,7 @@ func TestRun(t *testing.T) {
 			[]requestTest{
 				{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 				{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 				{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			},
 		)
@@ -327,7 +327,7 @@ func TestRun_Errors(t *testing.T) {
 			[]requestTest{
 				{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 				{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 			},
 		)
 
@@ -389,7 +389,7 @@ func TestRun_Errors(t *testing.T) {
 			[]requestTest{
 				{method: "GET", host: "discord.com", uri: "/api/v9/channels/channel-123/messages?limit=100"},
 				{method: "POST", host: "discord.com", uri: "/api/v9/channels/channel-123/messages",
-					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null}`},
+					body: `{"embeds":[{"type":"rich","title":"my title 1"}],"tts":false,"components":null,"sticker_ids":null}`},
 				{method: "PUT", host: "discord.com", uri: "/api/v9/channels/channel-123/messages/123/reactions/my-emoji-1:emoji-123/@me"},
 			},
 		)

Original file line number	Diff line number	Diff line change
`@@ -172,8 +172,6 @@ func TestReadConfiguration_Errors(t *testing.T) {`
`172`	`172`	`}`
`173`	`173`
`174`	`174`	`for testCaseName, testCase := range testCases {`
`175`		`- testCaseName, testCase := testCaseName, testCase`
`176`		`-`
`177`	`175`	`t.Run(testCaseName, func(tt *testing.T) {`
`178`	`176`	`tt.Parallel()`
`179`	`177`
Original file line number	Diff line number	Diff line change
`@@ -89,8 +89,6 @@ func TestNewHealthchecksManager_ErrorHasValidConfigurationInFile(t *testing.T) {`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`for testCaseName, testCase := range testCases {`
`92`		`- testCaseName, testCase := testCaseName, testCase`
`93`		`-`
`94`	`92`	`t.Run(testCaseName, func(tt *testing.T) {`
`95`	`93`	`var bufferLogs bytes.Buffer`
`96`	`94`	`log.Logger = zerolog.New(&bufferLogs).Level(zerolog.TraceLevel).With().Logger()`