Skip to content

docs: improve narrative flow, correct mistakes, and fix heading levels - #1172

Closed
dem-extra1 wants to merge 2 commits into
mainfrom
improve-narrative-flow-and-fix-headings-7702858823191244618
Closed

dem-extra1 wants to merge 2 commits into
mainfrom
improve-narrative-flow-and-fix-headings-7702858823191244618

Conversation

@dem-extra1

Copy link
Copy Markdown
Collaborator
  • Fix inner heading levels (### -> ####) inside theorem and definition divs to prevent document structure disruption.
  • Clarify bare demonstratives and improve narrative flow across subfiles.
  • Correct typos, spatial references, and subject-verb agreement in syllabus, survival models, and simulation app chapters.

PR created automatically by Jules for task 7702858823191244618 started by @dem-extra1

Co-authored-by: dem-extra1 <112029334+dem-extra1@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Co-authored-by: dem-extra1 <112029334+dem-extra1@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Claude finished reviewView run


PR is confirmed open, not draft. This matches the authoritative status already provided. I've reviewed the delta since the last round.

Review

What changed since the last round: Only binom.likelihood.app.qmd (commit 0e96bb4), reflowing a single long paragraph into one sentence per source line. I diffed the file content at both commits (63625646 and 712c46af) directly: the prose text itself is byte-identical except for line breaks — "our data are generated" (the subject-verb fix from the prior round) is unchanged, and no words were added, removed, or reordered.

This reflow is a direct, correct application of the CLAUDE.md rule: "One source line per major phrase in prose — keeps git diffs readable and review easier." Each new line break falls at a natural sentence/clause boundary ("...as seen previously." / "Larger sample sizes..." / "...with larger sample sizes." / "We also see..." / "...5% of the time." / "In this set of simulations," / "our data are generated under the null hypothesis," / "so our ability..."), so the rendered Markdown is unaffected (soft line breaks collapse to spaces) — this is a source-only formatting improvement, not a content change.

I re-checked this against the "don't flag cosmetic source-only formatting that renders identically" guidance: this isn't a flag, it's confirmation the change is a genuine, harmless improvement consistent with the repo's stated diff-readability convention.

No LaTeX/math was touched in this delta, so there's nothing to check against latex-macros/macros.qmd for this round. No new factual claims, citations, or computed values were introduced. No hallucination risk in a pure line-wrap change.

The rest of the PR (all the ####### heading-level fixes and demonstrative-pronoun fixes across the various _subfiles/, plus the earlier typo/grammar fixes) is unchanged from the prior round and was already reviewed clean in the 2026-09-16T21:01:29Z summary (verdict: Ready for merge, commit 636256465a72689ddcd9d55b2a70a0732c17a285). That verdict still stands — no regressions or new issues introduced by this round's change.

Verdict

Ready for merge

Structured Review Data (JSON)
{
  "schema_version": "1.1",
  "reviewer": "claude",
  "commit_sha": "712c46a",
  "verdict": "CLEAN",
  "findings": [],
  "detailed_assessment": "The only change since the prior round is a paragraph reflow in binom.likelihood.app.qmd into one sentence per line, with no wording altered, verified via direct git show diff of both commits.",
  "holistic_assessment": "This reflow satisfies the repository's one-phrase-per-line prose convention for diff readability and introduces no rendering, scope, or regression risk beyond the previously confirmed clean verdict."
}

Reviewed commit: 0e96bb4

@github-actions

Copy link
Copy Markdown
Contributor

💰 Cost: $0.2289 (review) — run

@github-actions

github-actions Bot commented Sep 16, 2026

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

2026-09-17 17:56 UTC

Copy link
Copy Markdown
Owner

Closing as a duplicate. This PR is one of seven automated dem-extra1 (Jules) PRs that ran the same "improve narrative flow / clarify demonstrative referents / fix heading levels" sweep with heavy file and content overlap (#1172, #1171, #1168, #1167, #1166, #1164, #1163). Rather than driving all seven through review separately for the same defects, #1168 (the largest/most complete sweep) is being kept and driven to merge. Content this PR touches that #1168 doesn't cover is tracked in #1174.

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


Generated by Claude Code

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