Skip to content

chore(revive-linter): add revive linter configuration#5515

Merged
Skarlso merged 3 commits intoexternal-secrets:mainfrom
Lumexralph:olumide/configure-revive-linter
Nov 1, 2025
Merged

chore(revive-linter): add revive linter configuration#5515
Skarlso merged 3 commits intoexternal-secrets:mainfrom
Lumexralph:olumide/configure-revive-linter

Conversation

@Lumexralph
Copy link
Copy Markdown
Contributor

Problem Statement

According to this PR #3396 that was implemented, the revive linter wasn't configured because it had loads of issues to be fixed. This PR configures the revive linter and adds the last piece needed to complete the fix and configuration of the revive linter.

Related Issue

Fixes #3397

Proposed Changes

Creates revive linter configuration and fixes additional issues introduced by merged code after the previous fix.

Format

Please ensure that your PR follows the following format for the title:

feat(scope): add new feature
fix(scope): fix bug
docs(scope): update documentation
chore(scope): update build tool or dependencies
ref(scope): refactor code
clean(scope): provider cleanup
test(scope): add tests
perf(scope): improve performance
desig(scope): improve design

Where scope is optionally one of:

  • charts
  • release
  • testing
  • security
  • templating

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

Signed-off-by: Olumide Ogundele <olumideralph@gmail.com>
@github-actions github-actions bot added kind/chore Categorizes Pull Requests for chore activities (like bumping versions) area/revive-linter size/m labels Oct 27, 2025
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 1, 2025

@Skarlso Skarlso merged commit c639cc2 into external-secrets:main Nov 1, 2025
29 checks passed
alexlebens pushed a commit to alexlebens/infrastructure that referenced this pull request Nov 10, 2025
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [external-secrets](https://github.com/external-secrets/external-secrets) | major | `0.20.4` -> `1.0.0` |

---

### Release Notes

<details>
<summary>external-secrets/external-secrets (external-secrets)</summary>

### [`v1.0.0`](https://github.com/external-secrets/external-secrets/releases/tag/v1.0.0)

[Compare Source](external-secrets/external-secrets@v0.20.4...v1.0.0)

Image: `ghcr.io/external-secrets/external-secrets:v1.0.0`
Image: `ghcr.io/external-secrets/external-secrets:v1.0.0-ubi`
Image: `ghcr.io/external-secrets/external-secrets:v1.0.0-ubi-boringssl`

<!-- Release notes generated using configuration in .github/release.yml at main -->

#### What's Changed

##### General

- chore: release helm chart for v0.20.4 by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5502](external-secrets/external-secrets#5502)
- chore(actions): fix security alerts for github actions by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5504](external-secrets/external-secrets#5504)
- chore(release): fix incorrect shas from security alert pinning by [@&#8203;jakobmoellerdev](https://github.com/jakobmoellerdev) in [#&#8203;5512](external-secrets/external-secrets#5512)
- chore: update dependencies by [@&#8203;eso-service-account-app](https://github.com/eso-service-account-app)\[bot] in [#&#8203;5519](external-secrets/external-secrets#5519)
- fix: upgrade go version by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5529](external-secrets/external-secrets#5529)
- fix(charts): normalize certificate duration default value by [@&#8203;aslafy-z](https://github.com/aslafy-z) in [#&#8203;5497](external-secrets/external-secrets#5497)
- chore(revive-linter): add revive linter configuration by [@&#8203;Lumexralph](https://github.com/Lumexralph) in [#&#8203;5515](external-secrets/external-secrets#5515)
- feat: go module separation by [@&#8203;moolen](https://github.com/moolen) in [#&#8203;5498](external-secrets/external-secrets#5498)
- fix: update deps script to reflect new go mod structure by [@&#8203;moolen](https://github.com/moolen) in [#&#8203;5532](external-secrets/external-secrets#5532)
- fix: lint by [@&#8203;gusfcarvalho](https://github.com/gusfcarvalho) in [#&#8203;5536](external-secrets/external-secrets#5536)
- feat: dynamic target implementation for external secrets sources  by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5470](external-secrets/external-secrets#5470)
- fix: codeql should not rebuild the world on every run by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5540](external-secrets/external-secrets#5540)
- fix(onepassword): Reduce API usage by [@&#8203;sondrelg](https://github.com/sondrelg) in [#&#8203;5410](external-secrets/external-secrets#5410)
- feat(esoctl): adds bootstrap generator commands by [@&#8203;gusfcarvalho](https://github.com/gusfcarvalho) in [#&#8203;5539](external-secrets/external-secrets#5539)
- fix: adds missing externalsecret\_type validation on generatorRef by [@&#8203;gusfcarvalho](https://github.com/gusfcarvalho) in [#&#8203;5547](external-secrets/external-secrets#5547)
- chore(docs): fix the generic resource doc using the incorrect switch by [@&#8203;Skarlso](https://github.com/Skarlso) in [#&#8203;5548](external-secrets/external-secrets#5548)
- docs(adopters): add Topicus.Education to adopters list by [@&#8203;sboschman](https://github.com/sboschman) in [#&#8203;5551](external-secrets/external-secrets#5551)
- fix(build): use more portable bash by [@&#8203;tiagolobocastro](https://github.com/tiagolobocastro) in [#&#8203;5537](external-secrets/external-secrets#5537)

##### Dependencies

- chore(deps): bump alpine from `4bcff63` to `4b7ce07` in /hack/api-docs by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5507](external-secrets/external-secrets#5507)
- chore(deps): bump golang from 1.25.2 to 1.25.3 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5506](external-secrets/external-secrets#5506)
- chore(deps): bump golang from `aee43c3` to `aee43c3` by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5516](external-secrets/external-secrets#5516)
- chore(deps): bump golang from 1.25.2-bookworm to 1.25.3-bookworm in /e2e by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5505](external-secrets/external-secrets#5505)
- chore(deps): bump anchore/sbom-action from 0.20.8 to 0.20.9 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5508](external-secrets/external-secrets#5508)
- chore(deps): bump hashicorp/setup-terraform from [`c529327`](external-secrets/external-secrets@c529327) to [`982f6f0`](external-secrets/external-secrets@982f6f0) by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5510](external-secrets/external-secrets#5510)
- chore(deps): bump actions/github-script from 7.0.1 to 8.0.0 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5513](external-secrets/external-secrets#5513)
- chore(deps): bump github/codeql-action from 4.30.9 to 4.31.0 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5514](external-secrets/external-secrets#5514)
- chore(deps): bump actions/cache from 4.2.4 to 4.3.0 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5511](external-secrets/external-secrets#5511)
- chore(deps): bump regex from 2025.9.18 to 2025.10.23 in /hack/api-docs by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5509](external-secrets/external-secrets#5509)
- chore(deps): bump mkdocs-macros-plugin from 1.4.0 to 1.4.1 in /hack/api-docs by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5518](external-secrets/external-secrets#5518)
- chore(deps): bump termcolor from 3.1.0 to 3.2.0 in /hack/api-docs by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5517](external-secrets/external-secrets#5517)
- chore(deps): bump github/codeql-action from 4.31.0 to 4.31.2 by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5531](external-secrets/external-secrets#5531)
- chore(deps): bump mkdocs-material from 9.6.22 to 9.6.23 in /hack/api-docs by [@&#8203;dependabot](https://github.com/dependabot)\[bot] in [#&#8203;5533](external-secrets/external-secrets#5533)

#### New Contributors

- [@&#8203;sondrelg](https://github.com/sondrelg) made their first contribution in [#&#8203;5410](external-secrets/external-secrets#5410)
- [@&#8203;tiagolobocastro](https://github.com/tiagolobocastro) made their first contribution in [#&#8203;5537](external-secrets/external-secrets#5537)

**Full Changelog**: <external-secrets/external-secrets@v0.20.4...v1.0.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 has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi4wLjEiLCJ1cGRhdGVkSW5WZXIiOiI0Mi4wLjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbImNoYXJ0Il19-->

Reviewed-on: https://gitea.alexlebens.dev/alexlebens/infrastructure/pulls/1985
Co-authored-by: Renovate Bot <renovate-bot@alexlebens.net>
Co-committed-by: Renovate Bot <renovate-bot@alexlebens.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/revive-linter kind/chore Categorizes Pull Requests for chore activities (like bumping versions) size/m

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Configure revive properly

2 participants