Skip to content

chore: make sure we are only building stripped go binaries for Docker containers#4382

Merged
mnafees merged 1 commit into
mainfrom
nafees/stripped-docker-go
Jul 9, 2026
Merged

chore: make sure we are only building stripped go binaries for Docker containers#4382
mnafees merged 1 commit into
mainfrom
nafees/stripped-docker-go

Conversation

@mnafees

@mnafees mnafees commented Jul 8, 2026

Copy link
Copy Markdown
Member

Description

Make sure we always strip symbols and debug information when building Go binaries.

Type of change

  • Chore (changes which are not directly related to any business logic)

Checklist

Changes have been:

  • Tested (unit, integration, or manually with steps specified)

🤖 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: [e.g. generating tests, writing docs]

@mnafees mnafees self-assigned this Jul 8, 2026
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hatchet-docs Ready Ready Preview, Comment Jul 8, 2026 4:55pm

Request Review

@mnafees mnafees requested a review from gregfurman July 8, 2026 17:01
@mnafees mnafees merged commit a0e424a into main Jul 9, 2026
36 checks passed
@mnafees mnafees deleted the nafees/stripped-docker-go branch July 9, 2026 10:37
mnafees added a commit that referenced this pull request Jul 9, 2026
mnafees added a commit that referenced this pull request Jul 9, 2026
* initial commit for no-auth local mode

* no-auth authz

* CLI and docs

* fix build error

* error boundary

* documentation changes

* local keyset

* no auth keyset in hatchet lite + docs

* rm stale file

* edit doc

* no auth user default to seed admin

* fmt docs

* simple test for noauth tokens

* auth disabled go build tag

* dev image

* rename strings

* rename files

* update docs

* prose rewording

* upload authmode disabled keyset

* format

* remove code

* fix precommitlint

* fix banner

* UI changes for no auth mode

* embed the JWT itself for auth disabled mode

* sane expiry

* PR comments

* test for noauth

* auth disabled test

* PR comments

* ignore noauth keysets and jwt for gitguardian

* resolve PR comments

* chore: make sure we are only building stripped go binaries for Docker containers (#4382)

* update docs

* update docs and build dashboard-dev images as well

* build all 5 images authdisabled in PR CI
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.

2 participants