Skip to content

Say when code belongs in a protocol - #48

Merged
lwaldron merged 1 commit into
mainfrom
authoring-code-guidance
Oct 18, 2026
Merged

lwaldron merged 1 commit into
mainfrom
authoring-code-guidance

Conversation

@lwaldron

Copy link
Copy Markdown
Member

AUTHORING.md said prose, not code, and left it there. Reviewing #46 and #47 showed the gap: "Ward linkage" does not say which of the two Ward variants in hclust() is meant, and naming the CBEA function for each inference step was clearer than describing it. This adds a short section saying code is welcome as a named call with its arguments written out, or as a formula, and that the prose must still be complete with every code block deleted.

Drafted with Claude Code.

Prose defines the operation; named library calls with their arguments
written out, and short formulae, pin the implementation. Add the test
that the prose must stand with every code block deleted.

Drafted with Claude Code.
Copilot AI balanced review requested due to automatic review settings October 18, 2026 02:53

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization has no extra usage available to pay for this review.

If your organization's extra usage balance is empty, an organization admin can add extra usage credits at claude.ai/admin-settings/usage. If its monthly spend limit was reached, an admin can raise it on the same page. If neither applies, contact Anthropic support.

Once extra usage is available, someone with write access to this repository can comment @claude review on this pull request to trigger a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused documentation changes clarify authoring guidance without introducing blocking issues.

No open issues 🎉

What changed in this PR

Clarifies when code belongs in protocols while keeping prose sufficient to describe the full procedure.

Changes:

  • Allows explicit library calls, formulas, and short pseudocode.
  • Defines code to omit and reinforces prose completeness in the author checklist.
File Description
AUTHORING.md Adds code-use guidance, examples, and a checklist clarification.

Review effort: Balanced

@lwaldron
lwaldron merged commit b6a7e35 into main Oct 18, 2026
2 checks passed
@lwaldron
lwaldron deleted the authoring-code-guidance branch October 18, 2026 02: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.

2 participants