Skip to content

python_oop: restore the French non-breaking spaces - #92

Open
mmcky wants to merge 1 commit into
mainfrom
typography/python_oop-nbsp
Open

mmcky wants to merge 1 commit into
mainfrom
typography/python_oop-nbsp

Conversation

@mmcky

@mmcky mmcky commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Closes #51.

Restores the ten non-breaking spaces before : ; ! ? in lectures/python_oop.md. The edit of 2026-10-02 (f2799d3), made in GitHub's web editor, replaced them with ordinary spaces; nothing else changes here.

The corrections in f2799d3 were checked against the English. All ten improve the French: Java, probabilités de transition, the spend method, the sentence on self, quad as code, application logistique, a comma, constitue, à l'aléa près and Dérivent. Two of them depart from the English where the English itself is wrong:

  • the English says spend "returns an error" when funds are insufficient, but its code prints a message, which is what the French now says;
  • the English writes "JAVA".

Both are being reported to QuantEcon/lecture-python-programming, so that the English is corrected and the two texts match again.

Assisted-by: Claude Code (Claude Opus 5.5)

🤖 Generated with Claude Code

… lines)

The 2026-10-02 edit (f2799d3) made in GitHub's web editor replaced them with ordinary spaces. Only those ten spaces change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 6, 2026 06:06
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-92--verdant-toffee-3261a8.netlify.app

Commit: 2e3f0b0

📚 Changed Lectures


Build Info

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

It is a verified whitespace-only restoration of the documented French non-breaking-space convention with no content changes and no detectable issues.

Review effort: Balanced
Findings: None

What changed in this PR

This PR restores ten French non-breaking spaces (U+00A0) before the punctuation : ; ! ? in lectures/python_oop.md. These were inadvertently replaced with ordinary spaces by an earlier GitHub web-editor edit (f2799d3), which is invisible in the web UI. The change is typography-only and contains no prose/content edits, keeping the French translation consistent with the repo-wide convention and preserving correct section-by-section sync with the English source. It closes issue #51.

Changes:

  • Replace the ordinary space with a non-breaking space before :/? on ten prose and heading lines.
  • No visible-character or content changes; only the whitespace byte changes.
File Description
lectures/​python_oop.md Restores NBSP (U+00A0) before French high punctuation on 10 lines reverted by a prior web-editor edit

I verified the raw bytes: each changed line now has C2 A0 (U+00A0) placed immediately before the punctuation mark (e.g. suivantes␣:, utile␣?), with a normal space following, and the diff carries no other content changes. A scan confirmed this matches the repo-wide French typography convention used across the lecture files. The only remaining regular-space-before-colon in the file is the title index on line 38, which is outside this PR's changed region and out of scope for the restoration.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mmcky mmcky mentioned this pull request Oct 6, 2026

This branch has not been deployed

No deployments
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.

Review: python_oop.md

2 participants