build(deps): bump github.com/getsentry/sentry-go from 0.23.0 to 0.24.0

Bumps [github.com/getsentry/sentry-go](https://github.com/getsentry/sentry-go) from 0.23.0 to 0.24.0.
- [Release notes](https://github.com/getsentry/sentry-go/releases)
- [Changelog](https://github.com/getsentry/sentry-go/blob/master/CHANGELOG.md)
- [Commits](https://github.com/getsentry/sentry-go/compare/v0.23.0...v0.24.0)

.gitlab-ci.yml

@@ -1,7 +1,7 @@
 include:
 - template: 'Workflows/MergeRequest-Pipelines.gitlab-ci.yml'
 
-image: golang:1.20.2
+image: golang:1.21.0
 
 stages:
 - deps
@@ -42,7 +42,7 @@ test:
 
 vulnerabilities:
   stage: test
-  image: golang:1.20.2
+  image: golang:1.21.0
   script:
   - go install golang.org/x/vuln/cmd/govulncheck@latest
   - govulncheck ./...
@@ -53,10 +53,10 @@ prepare_release:
   before_script:
   - npm install -g conventional-changelog-cli
   script:
-  - echo "DESCRIPTION=$(conventional-changelog -p conventionalcommits)" > variables.env
+  - conventional-changelog -p conventionalcommits > ./release.description
   artifacts:
-    reports:
-      dotenv: variables.env
+    paths:
+    - release.description
   rules:
   - if: $CI_COMMIT_TAG
 
@@ -70,6 +70,6 @@ release:
   - echo "Running release_job for $TAG"
   release:
     tag_name: '$CI_COMMIT_TAG'
-    description: '$DESCRIPTION'
+    description: ./release.description
   rules:
   - if: $CI_COMMIT_TAG

.golangci.yml

@@ -0,0 +1,2 @@
+run:
+  allow-parallel-runners: true

.pre-commit-config.yaml

@@ -10,16 +10,15 @@ repos:
     args:
     - --allow-multiple-documents
   - id: check-added-large-files
-- repo: https://github.com/jumanjihouse/pre-commit-hooks
-  rev: 3.0.0
-  hooks:
-  - id: markdownlint
 - repo: https://gitlab.com/devopshq/gitlab-ci-linter
-  rev: v1.0.3
+  rev: v1.0.5
   hooks:
   - id: gitlab-ci-linter
+    args:
+    - --project
+    - unboundsoftware/shiny/sentrysetup
 - repo: https://github.com/alessandrojcm/commitlint-pre-commit-hook
-  rev: v9.4.0
+  rev: v9.5.0
   hooks:
   - id: commitlint
     stages: [ commit-msg ]
@@ -36,5 +35,8 @@ repos:
   rev: v0.0.1
   hooks:
   - id: go-test
-  - id: golangci-lint
   - id: gofumpt
+- repo: https://github.com/golangci/golangci-lint
+  rev: v1.54.2
+  hooks:
+  - id: golangci-lint

go.mod

@@ -4,8 +4,8 @@ go 1.19
 
 require (
 	github.com/apex/log v1.9.0
-	github.com/getsentry/sentry-go v0.20.0
-	github.com/stretchr/testify v1.8.2
+	github.com/getsentry/sentry-go v0.24.0
+	github.com/stretchr/testify v1.8.4
 	gitlab.com/unboundsoftware/apex-mocks v0.2.0
 )
 

go.sum

@@ -10,8 +10,8 @@ github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/fatih/color v1.7.0/go.mod h1:Zm6kSWBoL9eyXnKyktHP6abPY2pDugNf5KwzbycvMj4=
 github.com/fsnotify/fsnotify v1.4.7/go.mod h1:jwhsz4b93w/PPRr/qN1Yymfu8t87LnFCMoQvtojpjFo=
-github.com/getsentry/sentry-go v0.20.0 h1:bwXW98iMRIWxn+4FgPW7vMrjmbym6HblXALmhjHmQaQ=
-github.com/getsentry/sentry-go v0.20.0/go.mod h1:lc76E2QywIyW8WuBnwl8Lc4bkmQH4+w1gwTf25trprY=
+github.com/getsentry/sentry-go v0.24.0 h1:02b7qEmJ56EHGe9KFgjArjU/vG/aywm7Efgu+iPc01Y=
+github.com/getsentry/sentry-go v0.24.0/go.mod h1:lc76E2QywIyW8WuBnwl8Lc4bkmQH4+w1gwTf25trprY=
 github.com/go-errors/errors v1.4.2 h1:J6MZopCL4uSllY1OfXM374weqZFFItUbrImctkmUxIA=
 github.com/go-logfmt/logfmt v0.4.0/go.mod h1:3RMwSq7FuexP4Kalkev3ejPJsZTpXXBr9+V4qmtdjCk=
 github.com/golang/protobuf v1.2.0/go.mod h1:6lQm79b+lXiMfvg/cZm0SGofjICqVBUtrP5yJMmIC1U=
@@ -46,15 +46,11 @@ github.com/smartystreets/assertions v1.0.0/go.mod h1:kHHU4qYBaI3q23Pp3VPrmWhuIUr
 github.com/smartystreets/go-aws-auth v0.0.0-20180515143844-0c1422d1fdb9/go.mod h1:SnhjPscd9TpLiy1LpzGSKh3bXCfxxXuqd9xmQJy3slM=
 github.com/smartystreets/gunit v1.0.0/go.mod h1:qwPWnhz6pn0NnRBP++URONOVyNkPyr4SauJk4cUOwJs=
 github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
-github.com/stretchr/objx v0.4.0/go.mod h1:YvHI0jy2hoMjB+UWwv71VJQ9isScKT/TqJzVSSt89Yw=
-github.com/stretchr/objx v0.5.0/go.mod h1:Yh+to48EsGEfYuaHDzXPcE3xhTkx73EhmCGUpEOglKo=
 github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
 github.com/stretchr/testify v1.6.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
-github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
 github.com/stretchr/testify v1.7.2/go.mod h1:R6va5+xMeoiuVRoj+gSkQ7d3FALtqAAGI1FQKckRals=
-github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
-github.com/stretchr/testify v1.8.2 h1:+h33VjcLVPDHtOdpUCuF+7gSuG3yGIftsP1YvFihtJ8=
-github.com/stretchr/testify v1.8.2/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o6fzry7u4=
+github.com/stretchr/testify v1.8.4 h1:CcVxjf3Q8PM0mHUKJCdn+eZZtm5yQwehR5yeSVQQcUk=
+github.com/stretchr/testify v1.8.4/go.mod h1:sz/lmYIOXD/1dqDmKjjqLyZ2RngseejIcXlSw2iwfAo=
 github.com/tj/assert v0.0.0-20171129193455-018094318fb0/go.mod h1:mZ9/Rh9oLWpLLDRpvE+3b7gP/C2YyLFYxNmcLnPTMe0=
 github.com/tj/assert v0.0.3 h1:Df/BlaZ20mq6kuai7f5z2TvPFiwC3xaWJSDQNiIS3Rk=
 github.com/tj/assert v0.0.3/go.mod h1:Ne6X72Q+TB1AteidzQncjw9PabbMp4PBMZ1k+vd1Pvk=

setup.go

@@ -17,8 +17,8 @@ var sentryInit = sentry.Init
 
 func SetupSentry(logger log.Interface, releaseName string, args SentryConfig) error {
 	if args.Disabled {
-		logger.Info("Sentry disabled, skipping setup")
-		return nil
+		logger.Info("Sentry disabled, setup using empty options")
+		return sentryInit(sentry.ClientOptions{})
 	}
 	if args.Environment == "" {
 		return fmt.Errorf("no Sentry environment supplied, exiting")

setup_test.go

@@ -34,7 +34,7 @@ func TestSetupSentry(t *testing.T) {
 			},
 			init:       nil,
 			wantErr:    assert.NoError,
-			wantLogged: []string{"info: Sentry disabled, skipping setup"},
+			wantLogged: []string{"info: Sentry disabled, setup using empty options"},
 		},
 		{
 			name: "no environment",