Skip to content

Skip redundant-version check in brew audit#7

Merged
kshahbw merged 1 commit into
mainfrom
fix/audit-skip-redundant-version
May 1, 2026
Merged

Skip redundant-version check in brew audit#7
kshahbw merged 1 commit into
mainfrom
fix/audit-skip-redundant-version

Conversation

@kshahbw
Copy link
Copy Markdown
Contributor

@kshahbw kshahbw commented May 1, 2026

Summary

GoReleaser regenerates Formula/band.rb on every release with an explicit version field. brew audit --strict flags this as redundant since the version can be parsed from the URL (/v0.1.0-beta/band_0.1.0-beta_*.tar.gz).

We've already hand-edited the formula to remove this line twice (commits 266a39f8, 032387d2) — but goreleaser re-emits it on every release, so the manual fix doesn't survive. The current goreleaser PR for v0.1.0-beta (#6) is failing audit for the same reason.

Pass --except=version to skip just the version-related checks while keeping the rest of the strict audit (description, license, URLs, style, etc.).

Test plan

GoReleaser regenerates Formula/band.rb on every release with an
explicit version field. brew audit --strict flags this as redundant
since the version can be parsed from the URL. The previous workaround
was to hand-edit each goreleaser PR, which does not survive the next
release.

Pass --except=version to skip just the version checks while keeping
the rest of the strict audit (description, license, URLs, etc.).
@kshahbw kshahbw requested review from a team as code owners May 1, 2026 19:13
@bwappsec
Copy link
Copy Markdown

bwappsec commented May 1, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@kshahbw kshahbw merged commit 06204e9 into main May 1, 2026
3 checks passed
@kshahbw kshahbw deleted the fix/audit-skip-redundant-version branch May 1, 2026 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants