Skip to content

SYNC: update short_path.md from lecture-python-intro - #28

Merged
mmcky merged 1 commit into
mainfrom
sync-short-path-from-source
Jun 21, 2026
Merged

mmcky merged 1 commit into
mainfrom
sync-short-path-from-source

Conversation

@mmcky

@mmcky mmcky commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Brings lectures/short_path.md back into byte-identical alignment with its primary source (see #7). Upstream landed a single-word fix:

```diff

  • The standard algorithm for finding $J$ is to start an initial guess and then iterate.
  • The standard algorithm for finding $J$ is to start with an initial guess and then iterate.
    ```

No intersphinx prefixes to preserve; short_path.md is not in #7's exception list.

🤖 Generated with Claude Code

Brings lecture-dp's short_path.md back into byte-identical alignment
with its primary source (see #7). Upstream landed a single-word fix:

  -  start an initial guess and then iterate.
  +  start with an initial guess and then iterate.

No intersphinx prefixes to preserve; short_path.md is not in #7's
exception list.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 21, 2026 22:46

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.

Pull request overview

This PR re-syncs lectures/short_path.md with its upstream primary source (lecture-python-intro) by applying the upstream wording correction noted in #7.

Changes:

  • Update one sentence to read “start with an initial guess” (instead of “start an initial guess”) to match upstream.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mmcky
mmcky merged commit 545575f into main Jun 21, 2026
2 checks passed
@mmcky
mmcky deleted the sync-short-path-from-source branch June 21, 2026 23:34
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