chore(deps): update module github.com/go-critic/go-critic to v0.14.0 (#7509)

This PR contains the following updates:

| Package | Change | Age | Confidence |
|---|---|---|---|
|
[github.com/go-critic/go-critic](https://redirect.github.com/go-critic/go-critic)
| `v0.13.0` -> `v0.14.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fgo-critic%2fgo-critic/v0.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fgo-critic%2fgo-critic/v0.13.0/v0.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>go-critic/go-critic (github.com/go-critic/go-critic)</summary>

###
[`v0.14.0`](https://redirect.github.com/go-critic/go-critic/releases/tag/v0.14.0)

[Compare
Source](https://redirect.github.com/go-critic/go-critic/compare/v0.13.0...v0.14.0)

#### What's Changed

- checkers/exitAfterDefer: handle optional `defer` properly by
[@&#8203;Afterous](https://redirect.github.com/Afterous) in
[#&#8203;1437](https://redirect.github.com/go-critic/go-critic/pull/1437)
- checkers/deprecatedComment: check that deprecated convention appears
in a separate paragraph by
[@&#8203;mx-psi](https://redirect.github.com/mx-psi) in
[#&#8203;1470](https://redirect.github.com/go-critic/go-critic/pull/1470)
- dev: enable goprintffuncname linter by
[@&#8203;alexandear](https://redirect.github.com/alexandear) in
[#&#8203;1478](https://redirect.github.com/go-critic/go-critic/pull/1478)
- checkers: add dupOption by
[@&#8203;alingse](https://redirect.github.com/alingse) in
[#&#8203;1471](https://redirect.github.com/go-critic/go-critic/pull/1471)
- all: rename gocritic to go-critic by
[@&#8203;cristaloleg](https://redirect.github.com/cristaloleg) in
[#&#8203;1482](https://redirect.github.com/go-critic/go-critic/pull/1482)
- deps: bump golang.org/x/tools from 0.31.0 to 0.32.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1484](https://redirect.github.com/go-critic/go-critic/pull/1484)
- checker/badCond: rhs should be also check for safe expr by
[@&#8203;codeimmortal](https://redirect.github.com/codeimmortal) in
[#&#8203;1486](https://redirect.github.com/go-critic/go-critic/pull/1486)
- deps: bump golang.org/x/tools from 0.32.0 to 0.33.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1489](https://redirect.github.com/go-critic/go-critic/pull/1489)
- checker: add `zeroByteRepeat` for avoiding `bytes.Repeat([]byte{0},
x)` by [@&#8203;codeimmortal](https://redirect.github.com/codeimmortal)
in
[#&#8203;1488](https://redirect.github.com/go-critic/go-critic/pull/1488)
- docs/readme: add Homebrew install instructions by
[@&#8203;krehel](https://redirect.github.com/krehel) in
[#&#8203;1491](https://redirect.github.com/go-critic/go-critic/pull/1491)
- deps: bump golang.org/x/tools from 0.33.0 to 0.34.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1492](https://redirect.github.com/go-critic/go-critic/pull/1492)
- deps: bump golang.org/x/tools from 0.34.0 to 0.35.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1498](https://redirect.github.com/go-critic/go-critic/pull/1498)
- ci: bump actions/checkout from 4 to 5 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1501](https://redirect.github.com/go-critic/go-critic/pull/1501)
- ci: bump actions/setup-go from 5 to 6 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1503](https://redirect.github.com/go-critic/go-critic/pull/1503)
- deps: bump golang.org/x/tools from 0.35.0 to 0.36.0 by
[@&#8203;dependabot](https://redirect.github.com/dependabot)\[bot] in
[#&#8203;1500](https://redirect.github.com/go-critic/go-critic/pull/1500)

#### New Contributors

- [@&#8203;mx-psi](https://redirect.github.com/mx-psi) made their first
contribution in
[#&#8203;1470](https://redirect.github.com/go-critic/go-critic/pull/1470)
- [@&#8203;codeimmortal](https://redirect.github.com/codeimmortal) made
their first contribution in
[#&#8203;1486](https://redirect.github.com/go-critic/go-critic/pull/1486)
- [@&#8203;krehel](https://redirect.github.com/krehel) made their first
contribution in
[#&#8203;1491](https://redirect.github.com/go-critic/go-critic/pull/1491)

**Full Changelog**:
<go-critic/go-critic@v0.13.0...v0.14.0>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/open-telemetry/opentelemetry-go).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNDMuMSIsInVwZGF0ZWRJblZlciI6IjQxLjE0My4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJTa2lwIENoYW5nZWxvZyIsImRlcGVuZGVuY2llcyJdfQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: dmathieu <[email protected]>

Author: renovate[bot]

internal/tools/go.mod

@@ -84,7 +84,7 @@ require (
 	github.com/fsnotify/fsnotify v1.9.0 // indirect
 	github.com/fzipp/gocyclo v0.6.0 // indirect
 	github.com/ghostiam/protogetter v0.3.17 // indirect
-	github.com/go-critic/go-critic v0.13.0 // indirect
+	github.com/go-critic/go-critic v0.14.0 // indirect
 	github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 // indirect
 	github.com/go-git/go-billy/v5 v5.6.2 // indirect
 	github.com/go-git/go-git/v5 v5.16.3 // indirect

internal/tools/go.sum

@@ -154,8 +154,8 @@ github.com/ghostiam/protogetter v0.3.17 h1:sjGPErP9o7i2Ym+z3LsQzBdLCNaqbYy2iJQPx
 github.com/ghostiam/protogetter v0.3.17/go.mod h1:AivIX1eKA/TcUmzZdzbl+Tb8tjIe8FcyG6JFyemQAH4=
 github.com/gliderlabs/ssh v0.3.8 h1:a4YXD1V7xMF9g5nTkdfnja3Sxy1PVDCj1Zg4Wb8vY6c=
 github.com/gliderlabs/ssh v0.3.8/go.mod h1:xYoytBv1sV0aL3CavoDuJIQNURXkkfPA/wxQ1pL1fAU=
-github.com/go-critic/go-critic v0.13.0 h1:kJzM7wzltQasSUXtYyTl6UaPVySO6GkaR1thFnJ6afY=
-github.com/go-critic/go-critic v0.13.0/go.mod h1:M/YeuJ3vOCQDnP2SU+ZhjgRzwzcBW87JqLpMJLrZDLI=
+github.com/go-critic/go-critic v0.14.0 h1:fzA7pGprRPwgA2LwoiaHuWzZzmUEM7dZjihfFpiZQpQ=
+github.com/go-critic/go-critic v0.14.0/go.mod h1:/2QgW4QJIJ9aYCYTQ9RWKc+1KnMFouAJ/FmPylhzOoU=
 github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 h1:+zs/tPmkDkHx3U66DAb0lQFJrpS6731Oaa12ikc+DiI=
 github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376/go.mod h1:an3vInlBmSxCcxctByoQdvwPiA7DTK7jaaFDBTtu0ic=
 github.com/go-git/go-billy/v5 v5.6.2 h1:6Q86EsPXMa7c3YZ3aLAQsMA0VlWmy43r6FHqa/UNbRM=

sdk/log/batch.go

@@ -375,25 +375,25 @@ func newBatchConfig(options []BatchProcessorOption) batchConfig {
 	c.maxQSize = c.maxQSize.Resolve(
 		clearLessThanOne[int](),
 		getenv[int](envarMaxQSize),
-		clearLessThanOne[int](),
+		clearLessThanOne[int](), // nolint:gocritic // the function argument is duplicated on purpose
 		fallback[int](dfltMaxQSize),
 	)
 	c.expInterval = c.expInterval.Resolve(
 		clearLessThanOne[time.Duration](),
 		getenv[time.Duration](envarExpInterval),
-		clearLessThanOne[time.Duration](),
+		clearLessThanOne[time.Duration](), // nolint:gocritic // the function argument is duplicated on purpose
 		fallback[time.Duration](dfltExpInterval),
 	)
 	c.expTimeout = c.expTimeout.Resolve(
 		clearLessThanOne[time.Duration](),
 		getenv[time.Duration](envarExpTimeout),
-		clearLessThanOne[time.Duration](),
+		clearLessThanOne[time.Duration](), // nolint:gocritic // the function argument is duplicated on purpose
 		fallback[time.Duration](dfltExpTimeout),
 	)
 	c.expMaxBatchSize = c.expMaxBatchSize.Resolve(
 		clearLessThanOne[int](),
 		getenv[int](envarExpMaxBatchSize),
-		clearLessThanOne[int](),
+		clearLessThanOne[int](), // nolint:gocritic // the function argument is duplicated on purpose
 		clampMax[int](c.maxQSize.Value),
 		fallback[int](dfltExpMaxBatchSize),
 	)