Conciseness: limit complex sentence structures (closes #30) - #36
Merged
Conversation
New section in conciseness.qmd (#30): compound sentences and the occasional dependent clause are fine, but several nested dependent clauses in one sentence make the reader hold each unfinished clause in mind. When a sentence gets hard to follow, split it into shorter sentences, each with one main idea. Includes a worked over-nested vs. split example. Matches the chapter's style; verified with `quarto render --to html`. Closes #30. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018v7h9ZVVQRBjdyRFGXCrzG
Resolve conflict with the convert-to-website restructure (#28): the new 'Limit complex sentence structures' section was added to the root conciseness.qmd, while main moved that file to chapters/conciseness.qmd. Git's rename detection carried the new section into chapters/conciseness.qmd; net change vs. main is the 27-line addition only. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BNqt1oGHGVxXjfgUiW6fLb
|
Claude finished @d-morrison's task in 1m 33s —— View job Code Review
Clean addition. Quarto syntax is correct ( No structural issues, no CI/build concerns, no accidental artifact commits. |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
The nested-clauses example's closing sentence credited the clarity gain to splitting alone, but the revision also restores chronological order (validation, then results, then decision) by lifting the main clause out from under two layers of subordination. Name the reordering so the explanation matches what the revision actually does. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018v7h9ZVVQRBjdyRFGXCrzG
|
Claude finished @d-morrison's task in 1m 29s —— View job Code Review
Clean. The previous inline finding (noting the reordering) has been addressed in commit No further issues found:
|
|
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
2 participants
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.
Summary
Adds a Limit complex sentence structures section to the Conciseness chapter (
conciseness.qmd), placed among the conciseness techniques (before the historical examples).Content (#30)
Note on the issue
#30 asked for examples "from open-access scientific literature (maybe our own)". I used a constructed example to demonstrate the principle cleanly (consistent with how the other chapters illustrate points, and avoiding sourcing/citation/link-rot overhead). Swapping in a real-literature before/after pair would be an easy follow-up if you'd prefer that.
Conventions followed
##section; semantic line breaks; Quarto#exm-example environment; no new external links. Verified withquarto render --to html.Closes
Closes #30.
🤖 Generated with Claude Code
Generated by Claude Code