Skip to content

fix: space Verso docstring closers in Section 1.2.2 - #621

Open
Chessing234 wants to merge 2 commits into
teorth:mainfrom
Chessing234:fix/section-1-2-2-verso-closer-spaces
Open

fix: space Verso docstring closers in Section 1.2.2#621
Chessing234 wants to merge 2 commits into
teorth:mainfrom
Chessing234:fix/section-1-2-2-verso-closer-spaces

Conversation

@Chessing234

@Chessing234 Chessing234 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • CI build green

teorth added a commit that referenced this pull request Aug 3, 2026
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@teorth

teorth commented Aug 3, 2026

Copy link
Copy Markdown
Owner

This now needs a small rebase: #632 merged and relabelled the third 1.2.22 docstring from (iii) to (ii'), so the Exercise 1.2.22(iii) (Product measure formula)-/ line this PR targets no longer exists.

The merged version already carries the closer space on that one line, so the resolution is just to drop that hunk. The other four lines here — 1.2.12(i), 1.2.12(ii), 1.2.22(i), 1.2.22(ii) — are untouched by #632 and still wanted.

Context on the relabel: Exercise 1.2.22 has only two parts in the text, with part (ii) reading "show that E × F is Lebesgue measurable, with m(E × F) = m(E) m(F)" — so the two theorems LebesgueMeasurable.prod and Lebesgue_measure.prod are the two halves of (ii). Same situation for Exercise 1.2.18.

Missing space before `-/` breaks the docstring terminator.
Leave the relabelled 1.2.22(ii') line alone — already spaced on main.
@Chessing234

Copy link
Copy Markdown
Contributor Author

rebased onto main and dropped the 1.2.22(iii)/(ii') hunk — the other four closers are still spaced.

@Chessing234
Chessing234 force-pushed the fix/section-1-2-2-verso-closer-spaces branch from 487c511 to 570f415 Compare August 4, 2026 13:45
@Chessing234

Copy link
Copy Markdown
Contributor Author

@teorth ready for another look when you have a minute — tip is rebased and the four closer spaces are still there.

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