Skip to content

docs: clarify install-from-source versioning and Claude Code requirement#254

Open
idrojone wants to merge 2 commits into
Gentleman-Programming:mainfrom
idrojone:docs/readme-clarifications
Open

docs: clarify install-from-source versioning and Claude Code requirement#254
idrojone wants to merge 2 commits into
Gentleman-Programming:mainfrom
idrojone:docs/readme-clarifications

Conversation

@idrojone
Copy link
Copy Markdown

Two documentation gaps addressed:

  1. Clarified that go install sets version to dev. Provided the correct go build -ldflags command for Windows and macOS/Linux to get the proper version stamp.
  2. Added a requirement note for Claude Code CLI >= 2.1.70 to support marketplace installation via git-subdir.

Closes #67

Copilot AI review requested due to automatic review settings April 26, 2026 20:00
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Collaborator

@Alan-TheGentleman Alan-TheGentleman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has the right approved issue link, so the process is close. I am not going to merge it until the PR has type:docs, is rebased onto main, and all required checks pass. Update those pieces and I will re-review.

- Updated README.md and INSTALLATION.md to explain go build vs go install versioning.
- Added minimum Claude Code CLI version (>= 2.1.70) to AGENT-SETUP.md.

Closes Gentleman-Programming#67
@idrojone idrojone force-pushed the docs/readme-clarifications branch from a5ee396 to 5e5bad9 Compare May 5, 2026 18:31
Copilot AI review requested due to automatic review settings May 5, 2026 18:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Windows, Linux, and other install methods → [docs/INSTALLATION.md](docs/INSTALLATION.md)

> [!TIP]
> Installing from source via `go build`? Use `-ldflags` to set the version string, or it will show as `"dev"`. See [Installation Guide](docs/INSTALLATION.md#install-from-source-macos--linux).
Comment thread docs/AGENT-SETUP.md

That's it. The plugin registers the MCP server, hooks, and Memory Protocol skill automatically.
> [!IMPORTANT]
> **Minimum Claude Code version:** Marketplace installation requires Claude Code CLI **>= 2.1.70**.
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.

docs: clarify install-from-source version stamp and Claude Code CLI minimum version

3 participants