Skip to content

Detangle phase3#47

Merged
ameowlia merged 5 commits into
mainfrom
detangle-phase3
Jun 18, 2026
Merged

Detangle phase3#47
ameowlia merged 5 commits into
mainfrom
detangle-phase3

Conversation

@geofffranks

Copy link
Copy Markdown
Contributor

Summary

Adds missing local testing scaffolding, fixes staticcheck issues

Backward Compatibility

Breaking Change? no

@geofffranks geofffranks requested a review from a team as a code owner June 18, 2026 18:32
geofffranks and others added 5 commits June 18, 2026 15:49
Co-authored-by: Cursor <cursoragent@cursor.com>
…ker, bin/test)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace dot gomega import with named import in cmd/locket/testrunner/runner.go.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Running with -d alone causes bash to exit immediately (no TTY/stdin),
which removes the container via --rm before the test runner can attach.
Adding -t allocates a pseudo-TTY so bash stays alive.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ameowlia ameowlia merged commit 193b22a into main Jun 18, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Application Runtime Platform Working Group Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants