Skip to content

Bump latex-macros to add \truemean and \truemeanf - #337

Merged
d-morrison merged 2 commits into
mainfrom
claude/bump-macros-truemean
Oct 18, 2026
Merged

d-morrison merged 2 commits into
mainfrom
claude/bump-macros-truemean

Conversation

@d-morrison

Copy link
Copy Markdown
Contributor

Requested by Ezra · project thread

Part of #336

Before: the latex-macros submodule points at 6dbf272, which has neither \truemean nor \truemeanf.

After: it points at c14884b, the commit that merged macros#225. No page changes, since the pin is the only edit.

How: the gitlink moves from 6dbf272 to c14884b; no other file changes. The rename of the old macros comes in a separate content PR stacked on this one.


Generated by Claude Code

Part of #336.

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

This comment has been minimized.

@github-actions

This comment has been minimized.

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

Copy link
Copy Markdown
Contributor

Claude finished review — View run


I found no problems in this PR. The only change against base is the latex-macros submodule pointer, which moves from b48fd22 to c14884b.

This confirms the prior review of 2026-10-18, which said Ready for merge at e54627c. That verdict still stands. The only change since then is a merge of main (commit 6bfbe4f), and that merge matches git's automatic merge of its parents. Dependabot-style submodule bumps are normal in this repo.

I read the diff but did not inspect the new submodule commit. I did not check which macros it defines or whether the site still renders with them.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "239333dd434905f4321718235b196bfbbc8552e8",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The only changed path against base is the latex-macros submodule pointer, now at c14884b, and the merge commit matches git's automatic merge.",
  "holistic_assessment": "Scope is a routine submodule bump consistent with the repository's dependency policy; I did not inspect the new submodule commit or render the site, so regression risk there is unchecked."
}

Reviewed commit: 239333d

@github-actions

Copy link
Copy Markdown
Contributor

💰 Cost: $0.1337 (review) — run

@github-actions

github-actions Bot commented Oct 18, 2026 •

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

2026-10-18 18:43 UTC

@d-morrison
d-morrison merged commit 4dae8ac into main Oct 18, 2026
23 checks passed
@d-morrison
d-morrison deleted the claude/bump-macros-truemean branch October 18, 2026 18:43
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