Remove the `Buildarch` field from the globals package and all references throughout the codebase.
**Changes:**
- Removed `Buildarch` package-level var and struct field from `internal/globals/globals.go`
- Removed `Buildarch` from the `New()` constructor
- Removed `globals.Buildarch = runtime.GOARCH` and unused `runtime` import from `cmd/webhooker/main.go`
- Removed `buildarch` from logger startup output in `internal/logger/logger.go`
- Removed all `Buildarch` test setup and assertions from globals, logger, database, and webhook_db_manager tests
All tests pass, `make check` passes, `docker build .` succeeds.
closes [issue #30](https://git.eeqj.de/sneak/webhooker/issues/30)
<!-- session: agent:sdlc-manager:subagent:5cae6803-6bdf-467d-9a56-43f135521e5f -->
Remove the Buildarch field from the globals package variable, struct,
and constructor. Remove all references in main.go (including the
runtime import), logger startup output, and test setup across globals,
logger, database, and webhook_db_manager packages.
closes #30
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Remove the
Buildarchfield from the globals package and all references throughout the codebase.Changes:
Buildarchpackage-level var and struct field frominternal/globals/globals.goBuildarchfrom theNew()constructorglobals.Buildarch = runtime.GOARCHand unusedruntimeimport fromcmd/webhooker/main.gobuildarchfrom logger startup output ininternal/logger/logger.goBuildarchtest setup and assertions from globals, logger, database, and webhook_db_manager testsAll tests pass,
make checkpasses,docker build .succeeds.closes issue #30
✅ Review: PASS
Clean removal of
Buildarchfrom the codebase. Verified:grep -r "Buildarch\|buildarch\|BUILDARCH" --include="*.go"returns nothingruntimeimport correctly removed fromcmd/webhooker/main.godocker build .succeeds (which runsmake check— formatting, linting, tests)remove globals.Buildarch from codebaseAll 7 files correctly updated. No logic changes, just dead code removal as intended.
Labeling
merge-readyand assigning to @sneak for final merge.