Skip to content

ci: adopt shared Go lint workflow - #53

Closed
VinnyBarton wants to merge 2 commits into
mainfrom
ci/adopt-shared-lint-validation
Closed

VinnyBarton wants to merge 2 commits into
mainfrom
ci/adopt-shared-lint-validation

Conversation

@VinnyBarton

Copy link
Copy Markdown
Collaborator

Summary

  • replace duplicated checkout, Go setup, dependency download, and golangci-lint steps with the shared Go lint workflow
  • pin the shared workflow to merged commit 08214ced07c278eaed1c68fa459916e07f071553
  • preserve the existing Go module location and golangci-lint v2.8 line

Security impact

The caller grants only contents read. External action maintenance moves into the reviewed shared workflow.

Validation

  • Actionlint on the changed workflow: pass
  • git diff --check: pass

Follow-up

Repository-wide Actions policy validation is intentionally deferred. It currently exposes existing ShellCheck findings in legacy build, SBOM, ReARM, and release workflows. Those findings should be removed through the next migration PRs rather than suppressed.

@VinnyBarton

Copy link
Copy Markdown
Collaborator Author

Superseded by #54. The lint-only split cannot meet the all-checks-pass requirement because the legacy OSV workflow fails while exporting oversized job outputs. PR #54 includes this lint migration, replaces that broken workflow, and has a fully green check suite.

@VinnyBarton VinnyBarton closed this Oct 6, 2026
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.

1 participant