Skip to content

fix: set git identity in registry git helpers and tests for CI#401

Merged
RobertLD merged 2 commits intomainfrom
fix/ci-git-identity-git-test
Mar 16, 2026
Merged

fix: set git identity in registry git helpers and tests for CI#401
RobertLD merged 2 commits intomainfrom
fix/ci-git-identity-git-test

Conversation

@RobertLD
Copy link
Copy Markdown
Owner

Summary

  • Fixes the same CI git identity issue from fix: set git identity in registry sync test for CI #400, but in src/registry/git.ts and tests/unit/registry/git.test.ts
  • Source: createRegistryRepo and commitAndPush now use -c user.name/email flags
  • Test: execSync git commit calls pass GIT_AUTHOR/COMMITTER env vars

Test plan

  • npx vitest run tests/unit/registry/git.test.ts tests/unit/registry/sync.test.ts — 21/21 pass
  • CI passes on this PR

🤖 Generated with Claude Code

RobertLD and others added 2 commits March 16, 2026 13:16
The createBareRepo helper runs git commit without author identity,
which fails on CI runners that lack global git config.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The git.ts commit helpers and git.test.ts both run git commit without
author identity, failing on CI runners with no global git config.

- Source: use -c user.name/email flags in createRegistryRepo and commitAndPush
- Test: pass GIT_AUTHOR/COMMITTER env vars to execSync calls

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 16, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
libscope Ignored Ignored Mar 16, 2026 5:03pm

@RobertLD RobertLD merged commit 9ede32f into main Mar 16, 2026
7 of 9 checks passed
@RobertLD RobertLD deleted the fix/ci-git-identity-git-test branch March 16, 2026 17:07
RobertLD added a commit that referenced this pull request Mar 16, 2026
Same git identity fix as #400 and #401, applied to the 4 integration
test files under tests/integration/registry/.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
RobertLD added a commit that referenced this pull request Mar 16, 2026
Same git identity fix as #400 and #401, applied to the 4 integration
test files under tests/integration/registry/.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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