Skip to content

Correct stale test notes and complete layout in AGENTS.md - #37

Merged
arbourd merged 1 commit into
mainfrom
arbourd/agents.md
Oct 15, 2026
Merged

arbourd merged 1 commit into
mainfrom
arbourd/agents.md

Conversation

@arbourd

@arbourd arbourd commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Fixes two inaccurate statements in the Testing section and fills gaps found in an audit against the code.

  • gitConfigGlobalFixture is defined once per package, not in each _test.go; redefining it fails to compile.
  • Completion tests set GETPATH, HOME and PATH via completionEnv; only GIT_CONFIG_GLOBAL is inherited.
  • Layout now lists completions_test.go and .goreleaser.yml.
  • Data flow now shows the --complete path.
  • Notes that the clone ssh github test needs GitHub SSH auth locally.

The gitConfigGlobalFixture is defined once per package, and completion tests set GETPATH/HOME/PATH themselves. Also list completions_test.go, .goreleaser.yml, the --complete flow, and the SSH test caveat.
@arbourd
arbourd merged commit d47d070 into main Oct 15, 2026
3 checks passed
@arbourd
arbourd deleted the arbourd/agents.md branch October 15, 2026 14:58
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