Skip to content

DOC Sync six drifting Jupytext notebook pairs - #2449

Merged
Roman Lutz (romanlutz) merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-sync-jupytext-pairs
Aug 22, 2026
Merged

DOC Sync six drifting Jupytext notebook pairs#2449
Roman Lutz (romanlutz) merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-sync-jupytext-pairs

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

Six documentation notebooks had drifted from their paired Jupytext percent files. This made code cells inconsistent between rendered notebooks and reviewable Python sources, including missing markdown cell separators and imports in the WebSocket Copilot target example.

Synchronize each pair from its authoritative content while preserving existing notebook outputs and metadata. The WebSocket and Playwright pairs merge the valid imports and cell boundaries from both representations; the remaining pairs align wording or remove stray code-cell whitespace. Jupytext-equivalent metadata differences outside these six pairs are intentionally left unchanged to avoid generated churn.

Tests and Documentation

  • Ran a repository-wide uv run jupytext --diff audit to identify the six drifting pairs.
  • Confirmed uv run jupytext --diff <file.py> <file.ipynb> produces no diff for every affected pair.
  • Ran targeted uv run ruff check and uv run ruff format --check on the changed percent files.
  • Ran git diff --check.
  • Commit hooks passed documentation structure, notebook sanitation, Ruff, and Jupyter Ruff checks.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@hannahwestra25 hannahwestra25 self-assigned this Aug 21, 2026
Comment thread doc/code/targets/10_3_websocket_copilot_target.ipynb
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 22, 2026
Merged via the queue into microsoft:main with commit 949cf3f Aug 22, 2026
54 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-sync-jupytext-pairs branch August 22, 2026 01:51
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.

3 participants