Skip to content

fix: git safe.directory and install order in release workflow - #8

Merged
jesse23 merged 1 commit into
mainfrom
feat/basic-cli-support
Mar 22, 2026
Merged

fix: git safe.directory and install order in release workflow#8
jesse23 merged 1 commit into
mainfrom
feat/basic-cli-support

Conversation

@jesse23

@jesse23 jesse23 commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Description

Two fixes for release workflow inside `oven/bun:1.3.10-debian` container:

  1. Move `apt-get install git` before `actions/checkout@v4` — checkout needs git
  2. Add `git config --global --add safe.directory` after checkout — GitHub Actions containers run as a different user than the one that cloned the repo, causing git to reject the directory as unsafe

ADRs

Move apt-get install git before checkout and add git config --global --add safe.directory after checkout to fix 'not a git repository' error in container.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@jesse23
jesse23 merged commit d500a81 into main Mar 22, 2026
3 checks passed
jesse23 added a commit that referenced this pull request Mar 23, 2026
fix: git safe.directory and install order in release workflow
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