Skip to content

Fix stale docs: upgrade example, build_info note, CHAT compat version…#89

Merged
bernardladenthin merged 1 commit intomasterfrom
claude/review-upgrade-docs-Ocf5x
Apr 18, 2026
Merged

Fix stale docs: upgrade example, build_info note, CHAT compat version…#89
bernardladenthin merged 1 commit intomasterfrom
claude/review-upgrade-docs-Ocf5x

Conversation

@bernardladenthin
Copy link
Copy Markdown
Owner

…, test prompt

CLAUDE.md:

  • Remove brittle line number from GIT_TAG upgrade instructions; update example value to b8831 and upgrade example from b5016/b5022 to b8808/b8831; replace stale branch name with generic
  • Qualify "top 8 rows" claim: add note that CMakeLists.txt must also be reviewed for build-system-level breaks not visible in header diffs
  • Update common/common.h priority-table note: build_info was removed in b8831 (now llama_build_info() from build-info.h)

CHAT_INTEGRATION_SUMMARY.md:

  • Clarify the implementation was done at b8611 (not "current codebase")
  • Update Upstream Compatibility section header to reflect that chat functionality has been verified compatible through b8831

CLAUDE_TEST_GENERATION_PROMPT.md:

  • Add status note at the top: all tasks have been executed; re-running as-is will conflict with existing files
  • Fix LlamaOutputTest spec: testTextFromBytes with byte[] constructor does not exist; correct to testTextFromString with String constructor

https://claude.ai/code/session_01HJUr9LwrMNpT3FTwwzR6Gh

…, test prompt

CLAUDE.md:
- Remove brittle line number from GIT_TAG upgrade instructions; update
  example value to b8831 and upgrade example from b5016/b5022 to b8808/b8831;
  replace stale branch name with generic <your-branch>
- Qualify "top 8 rows" claim: add note that CMakeLists.txt must also be
  reviewed for build-system-level breaks not visible in header diffs
- Update common/common.h priority-table note: `build_info` was removed in
  b8831 (now `llama_build_info()` from `build-info.h`)

CHAT_INTEGRATION_SUMMARY.md:
- Clarify the implementation was done at b8611 (not "current codebase")
- Update Upstream Compatibility section header to reflect that chat
  functionality has been verified compatible through b8831

CLAUDE_TEST_GENERATION_PROMPT.md:
- Add status note at the top: all tasks have been executed; re-running
  as-is will conflict with existing files
- Fix LlamaOutputTest spec: `testTextFromBytes` with `byte[]` constructor
  does not exist; correct to `testTextFromString` with `String` constructor

https://claude.ai/code/session_01HJUr9LwrMNpT3FTwwzR6Gh
@bernardladenthin bernardladenthin merged commit 6690862 into master Apr 18, 2026
10 checks passed
@bernardladenthin bernardladenthin deleted the claude/review-upgrade-docs-Ocf5x branch April 18, 2026 12:35
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.

2 participants