Conversation
…555) Draft contributor guide covering dev-environment setup, Makefile targets, branch and PR conventions, tests-and-docs requirements, issue labels, the AI-agent policy, a copyright placeholder, security reporting, and a code of conduct. Draft-note blockquotes mark the decisions still open for maintainer review. Also: - CLAUDE.md: new section telling agents to read CONTRIBUTING.md and keep it current. - docs/wiki/README.md: index row for CONTRIBUTING.md. - docs/handbook/installation.html: build-from-source requirements said Go 1.22+ and omitted protoc; now Go 1.26+ per go.mod, and protoc is listed since the modem build script needs it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…hers fork and sync (GH #555) Spell out that write access to chrissnell/graywolf is maintainer-only, so every other contributor forks, keeps the fork's main synced with upstream, branches on the fork, and opens the PR from there. Adds the upstream-remote and fast-forward sync commands. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…triage process (GH #555) Adds support, needs-info, needs-discussion, needs-second-review, and awaiting-author as proposed labels, with the issue and PR triage flow maintainers will follow. Narrows the meaning of the existing question label. A draft note poses the plain-name vs status: prefix decision, which must be settled before the labels are created. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Owner
|
I will need to look this weekend. In general I would say:
- always use your personal GitHub account.
- never use a work account
- never ever allow commits from @claude etc or attribution to an AI agent
anywhere. Not in the code, not in the README, not in the commit message of
PR. Not with a dog, not on a log. Only humans.
- use obra/superpowers skills when authoring and be sure to use
requesting-code-review skill to review your code before PRing
- all new functions require proper tests
- new features should be cleanly documented in the handbook in plain, easy
to understand English.
…On Wed, Sep 23, 2026 at 21:39 kotfu ***@***.***> wrote:
@kotfu <https://github.com/kotfu> requested your review on:
chrissnell/graywolf#602 <#602>
docs: add CONTRIBUTING.md (GH #555
<#555>).
—
Reply to this email directly, view it on GitHub
<#602?email_source=notifications&email_token=AAIF34XDCLZABRF22JG3CGT5QSJNZA5CNFSNUABQM5UWIORPF5TWS5BNNB2WEL2JONZXKZKFOZSW45CON52GSZTJMNQXI2LPNYXTGMJXGI3DKNZXGI4TDJTSMVQXG33OWBZGK5TJMV3V64TFOF2WK43UMVSKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#event-31726577291>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIF34RXDYVMMH6YRAIKCCT5QSJNZAVCNFSNUABGKJSXA33TNF2G64TZHMYTCOJYGAYDAOBTGU5US43TOVSTWNJVGYZDMOJXGM2DBILWAI>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #555.
@chrissnell -- this is the CONTRIBUTING.md draft you asked for in #555. Could you give it a review? There are a handful of policy calls in it that are yours to make, listed below. I'd rather settle them here than guess.
What's in it
CONTRIBUTING.md(new). Covers:It's written so agents can follow it too, per your comment on Draft a CONTRIBUTING.md for review and adoption #555, and it points at
docs/wiki/for architecture.CLAUDE.md: a short section pointing agents at CONTRIBUTING.md.docs/wiki/README.md: an index row for CONTRIBUTING.md.docs/handbook/installation.html: fixes the build-from-source requirements. Go 1.26+ (it said 1.22+;go.modisgo 1.26.2), and addsprotoc, which was missing even though the modem's build script needs it.Decisions needed before adoption
The draft marks each of these with a "Draft note for reviewers" blockquote. Those get removed once they're decided.
Signed-off-by, or aCo-authored-bytrailer (required, optional, or discouraged)? Or is "you are the author and responsible for it" enough?graywolf@nw5w.com, the contact on the handbook's privacy page. Is that the address you want?CODE_OF_CONDUCT.md? Either way, please confirm the reporting contact.support,needs-info,needs-discussion,needs-second-reviewandawaiting-author, plus the triage flow that uses them. Two sub-questions:in-progress,backlogandblocked-pending-other-work? Or astatus:prefix, which would also mean renaming those three?questionto design or behavior questions, and move support requests tosupportand missing-detail reports toneeds-info?Follow-ups, not in this PR
These are separate PRs or repo settings, and none of them block adoption. They're listed in detail in this comment on #555.
Testing
Docs only. The branch merges cleanly with current
main.🤖 Generated with Claude Code