Skip to content

Replace a hijacked generator link - #328

Merged
moaz-magdy merged 1 commit into
masterfrom
fix/hijacked-generator-link
Oct 27, 2026
Merged

moaz-magdy merged 1 commit into
masterfrom
fix/hijacked-generator-link

Conversation

@moaz-magdy

Copy link
Copy Markdown
Contributor

The "generator functions" link in these 2 notebooks went to jeffknupp.com. That domain now redirects to an
unrelated streaming and betting site, so learners who click it land there. The link now points to the Python
glossary entry for generators:

  • before: https://jeffknupp.com/blog/2013/04/07/improve-your-python-yield-and-generators-explained/
  • after: https://docs.python.org/3/glossary.html#term-generator

Nothing else changes: one line per notebook, and no cell outputs or metadata.

Testing

  • Each notebook still loads as JSON, and re-saving it in Jupyter's own format (indent=1, sorted keys,
    non-ASCII kept) gives back the exact file, so the diff is only the edited line.
  • git grep jeffknupp finds nothing after the change.
  • The new URL returns HTTP 200 and its #term-generator anchor exists (checked 2026-10-25).
  • No notebook was run: the change is a link in a markdown cell.

🤖 Generated with Claude Code

jeffknupp.com now redirects to an unrelated streaming and betting site. Link to the Python glossary entry for generators instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@moaz-magdy
moaz-magdy requested a review from a team as a code owner October 26, 2026 15:01
@moaz-magdy
moaz-magdy requested review from SudKul and removed request for a team October 26, 2026 15:01
@moaz-magdy
moaz-magdy merged commit dd93380 into master Oct 27, 2026
1 check failed
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.

1 participant