FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add CLAUDE.md by jamiew · Pull Request #137 · git-friendly/git-friendly · GitHub

Add CLAUDE.md - #137

Closed
jamiew wants to merge 1 commit into
mainfrom
add-claude-md
Closed

Add CLAUDE.md#137
jamiew wants to merge 1 commit into
mainfrom
add-claude-md

Conversation

jamiew commented Jun 1, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

short notes for AI agents working on the repo. What they need to know that they can't get from reading the code.

mostly:

  • where the test suite lives -- it's the ci.yml workflow, replayed locally against a temp fixture
  • bash conventions worth following in this codebase -- modern git plumbing, no parsing porcelain, quote vars, locale-safe
  • pointer to the global style rules for commits / PRs / branch names

informed by the modernize-git-commands (#128) and add-test-coverage work

The "don't parse git branch output" and "set +H in test wrappers" notes are lessons learned the hard way during that audit.

short notes for AI agents working on the repo: how to run the test suite locally (replay ci.yml against a $TMPDIR fixture), the bash conventions worth following (plumbing over porcelain, git switch over checkout, quote everything, locale-safe), and a pointer to the global style rules for commits/PRs.
jamiew closed this Jun 2, 2026
jamiew deleted the add-claude-md branch June 2, 2026 19:18
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL