Skip to content

Cite aliases directly instead of saying who uses the name - #16

Merged
dem-extra1 merged 1 commit into
mainfrom
claude/project-thread-dspcrc
Oct 15, 2026
Merged

dem-extra1 merged 1 commit into
mainfrom
claude/project-thread-dspcrc

Conversation

@dem-extra1

Copy link
Copy Markdown
Contributor

Requested by Ezra · project thread

Before: the Hadamard code definition said "Arora and Barak call it the Walsh--Hadamard code."

After: "Also called the Walsh--Hadamard code [@arora2009computational, chapter 11]."

How: the citation already says the source uses the name. Not rendered.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CZKcRKHQWBS3kkHseDyiPr


Generated by Claude Code

A citation after a bold alias already says the source uses that name.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CZKcRKHQWBS3kkHseDyiPr
@github-actions

Copy link
Copy Markdown

Claude finished review — View run


Review of PR #16 (commit 7b98701)

The diff changes one line in _subfiles/_def-hadamard-code.qmd. The old line was @arora2009computational [chapter 11] call it the Walsh--Hadamard code. The new line is Also called the **Walsh--Hadamard code** [@arora2009computational, chapter 11].

I found no blocking issues.

  • Citation: the key arora2009computational exists in references.bib. The bracketed [@key, chapter 11] form is valid Pandoc citation syntax.
  • Definition div: the def-hadamard-code div is intact, and the #def- anchor is unchanged, so links from course sites keep working.
  • Alias: the second name is now a bolded alternate term. This fits the definitions checklist.
  • Optional: "Also called the …" is a sentence fragment. "It is also called the …" would read more cleanly. This is a style suggestion only.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "7b98701ad38e2fa723ff8a84a01d18324d9c26ea",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The one-line change in _subfiles/_def-hadamard-code.qmd uses valid citation syntax and the arora2009computational key exists in references.bib.",
  "holistic_assessment": "The change is tightly scoped, leaves the def-hadamard-code anchor intact so external links keep working, and introduces no regression or build risk."
}

Reviewed commit: 7b98701

@github-actions

Copy link
Copy Markdown

💰 Cost: $0.1360 (review) — run

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.

2026-10-15 21:44 UTC

@dem-extra1

Copy link
Copy Markdown
Contributor Author

Cross-harness review (Argon: Gemini via Antigravity agy 1.3.5, read-only plan mode) at 7b98701: CLEAN

Checked for correctness, Quarto syntax and crossrefs, consistency with the PR description, lost or duplicated content, and style, from the diff alone. Not checked: the rendered page.

Posted by Claude Code (AI agent) --- not written by a human.

@dem-extra1
dem-extra1 merged commit 4195480 into main Oct 15, 2026
16 checks passed
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