Skip to content

HYPERFLEET-879 - feat: add bingo-aware Go tooling hooks#3

Open
rafabene wants to merge 1 commit intoopenshift-hyperfleet:mainfrom
rafabene:HYPERFLEET-879-bingo-aware-go-hooks
Open

HYPERFLEET-879 - feat: add bingo-aware Go tooling hooks#3
rafabene wants to merge 1 commit intoopenshift-hyperfleet:mainfrom
rafabene:HYPERFLEET-879-bingo-aware-go-hooks

Conversation

@rafabene
Copy link
Copy Markdown
Contributor

@rafabene rafabene commented Apr 8, 2026

Summary

  • Add pkg/bingo package for automatic detection of bingo-managed tool binaries in .bingo/
  • Add three new pre-commit hooks as Go subcommands: golangci-lint, gofmt, go-vet
  • Hooks detect bingo-pinned versions first, fall back to system-installed versions
  • Clear error messages when tools are not found or bingo binary is not built

Depends on: #2

Acceptance Criteria

  • Hooks detect bingo-managed tool versions and use them when available
  • Falls back to system-installed versions when bingo is not configured
  • Hooks cover: golangci-lint, gofmt, go vet
  • Clear error messages when tools are not found
  • Documentation for hook configuration in consuming repos

Test Plan

  • go test -race ./... passes (17 new tests, 53 total)
  • go vet ./... passes
  • gofmt -l . clean on new files
  • Bingo detection tested with mock .bingo/ directories (88.6% coverage on pkg/bingo)

@rafabene rafabene changed the base branch from HYPERFLEET-878-go-implementation to main April 8, 2026 17:16
@rafabene rafabene force-pushed the HYPERFLEET-879-bingo-aware-go-hooks branch from 65296e6 to b003bba Compare April 14, 2026 18:40
@rafabene rafabene marked this pull request as ready for review April 14, 2026 18:40
Add golangci-lint, gofmt, and go-vet pre-commit hooks that automatically
detect and use bingo-managed tool versions from .bingo/, falling back to
system-installed versions when bingo is not configured.
@rafabene rafabene force-pushed the HYPERFLEET-879-bingo-aware-go-hooks branch from b003bba to 143aa0c Compare April 14, 2026 18:51
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