Skip to content

gh-133403: Add type annotations to generate_levenshtein_examples.py - #143317

Merged
hugovk merged 4 commits into
python:mainfrom
sandole:add-typing-generate-levenshtein
Apr 18, 2026
Merged

gh-133403: Add type annotations to generate_levenshtein_examples.py#143317
hugovk merged 4 commits into
python:mainfrom
sandole:add-typing-generate-levenshtein

Conversation

@sandole

@sandolesandole commented Dec 31, 2025•
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Added type annotations to Tools/build/generate_levenshtein_examples.py
  • Updated Tools/build/mypy.ini to include the file for strict type checking
  • Updated .github/workflows/mypy.yml to check the file in CI

Type annotations added

  • Function parameters and return types for _substitution_cost, levenshtein, and main
  • Variable type annotation for the examples set
  • Added from __future__ import annotations for cleaner syntax

Test plan

  • Ran mypy --config-file=Tools/build/mypy.ini Tools/build/generate_levenshtein_examples.py
  • All mypy strict checks pass with no issues

This continues the ongoing effort in #133403 to add type checking to more build tools.

📚 Documentation preview 📚: https://cpython-previews--143317.org.readthedocs.build/

@bedevere-app

bedevere-appBot commented Dec 31, 2025

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-appbedevere-appBot mentioned this pull request Dec 31, 2025
Closed
JelleZijlstra
JelleZijlstra reviewed Dec 31, 2025
View reviewed changes
Comment threadDoc/library/typing.rst Outdated
JelleZijlstra
JelleZijlstra reviewed Dec 31, 2025
View reviewed changes
Comment threadTools/build/generate_levenshtein_examples.py Outdated
JelleZijlstra
JelleZijlstra reviewed Dec 31, 2025
View reviewed changes

@JelleZijlstraJelleZijlstra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not sure we're OK with adding typing to arbitrary files. I'd prefer to get an approval from someone who works on this file.

@AlexWaygood
AlexWaygood removed their request for review January 1, 2026 01:35
@sandole
sandoleforce-pushed the add-typing-generate-levenshtein branch from ce4462f to 22a495dCompareJanuary 1, 2026 02:51
@AA-Turner

AA-Turner commented Jan 1, 2026

Copy link
Copy Markdown
Member

I'd agree with Jelle, I also think we should weigh up churn cost for PRs in this category. This PR doesn't seem to be LLM generated at first glance, but I imagine this ('add type hints to X file') could be a magnet for such activity.

A

@AA-TurnerAA-Turner added the pending The issue will be closed if no feedback is provided label Jan 1, 2026
@sandole

sandole commented Jan 1, 2026

Copy link
Copy Markdown
ContributorAuthor

sorry guys first time contributor here but would really like to be a regular contributor

who's the go-to guy for these CI / tooling changes?

@picnixz

picnixz commented Jan 1, 2026

Copy link
Copy Markdown
Member

That would be any of the maintainer working on traceback.py. The original author's @ambv, but considering we actually didn't change this file since 2022, I think we won't ever have a change to change it in the future either.

Also, please avoid updating the branch if the CI is still green. It wastes CI resources as mentioned in https://devguide.python.org/getting-started/pull-request-lifecycle/#update-branch-button (more generally, please read the devguide).

vstinner
vstinner reviewed Jan 8, 2026
View reviewed changes
Comment threadMisc/NEWS.d/next/Tools-Demos/2025-12-31-10-30-00.gh-issue-133403.aB3xYz.rst Outdated
@hugovk
hugovk requested a review from webknjaz as a code ownerMarch 21, 2026 17:35
@bedevere-app

This comment was marked as resolved.

Sign in to view
@hugovk
hugovk removed their request for review April 6, 2026 12:57
@vstinner

vstinner commented Apr 7, 2026

Copy link
Copy Markdown
Member

cc @sobolevn

sobolevn
sobolevn approved these changes Apr 7, 2026
View reviewed changes

@sobolevnsobolevn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! Thank you!

@sobolevnsobolevn removed the pending The issue will be closed if no feedback is provided label Apr 7, 2026
@sobolevn

sobolevn commented Apr 7, 2026

Copy link
Copy Markdown
Member

In my opinion, this is a really small PR that add pretty simple types. So, this is +1 from me :)

@hugovk
hugovk merged commit 28b8d5f into python:mainApr 18, 2026
60 checks passed
@hugovkhugovk added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Apr 18, 2026
@miss-islington-app

miss-islington-appBot commented Apr 18, 2026

Copy link
Copy Markdown

Thanks @sandole for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@miss-islington-app

miss-islington-appBot commented Apr 18, 2026

Copy link
Copy Markdown

Thanks @sandole for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 18, 2026
@sandole@hugovk@miss-islington
pythongh-133403: Add type annotations to generate_levenshtein_example…
f4718bd
…s.py (pythonGH-143317)
(cherry picked from commit 28b8d5f)
Co-authored-by: John Seong <39040639+sandole@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@miss-islington-app

miss-islington-appBot commented Apr 18, 2026

Copy link
Copy Markdown

Sorry, @sandole and @hugovk, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 28b8d5ffccd355dad7c8fd2fbf7b7552083c7e14 3.13

@bedevere-app

bedevere-appBot commented Apr 18, 2026

Copy link
Copy Markdown

GH-148734 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Apr 18, 2026
@hugovkhugovk removed the needs backport to 3.13 bugs and security fixes label Apr 18, 2026
hugovk added a commit that referenced this pull request Apr 19, 2026
@miss-islington@sandole@hugovk
[3.14]gh-133403: Add type annotations to generate_levenshtein_exampl…
b0104a9
…es.py (GH-143317) (#148734)
Co-authored-by: John Seong <39040639+sandole@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@sandole@hugovk@ljfp
pythongh-133403: Add type annotations to generate_levenshtein_example…
4eb08bb
…s.py (python#143317)
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner left review comments
@JelleZijlstraJelleZijlstraJelleZijlstra left review comments
@sobolevnsobolevnsobolevn approved these changes
@ezio-melottiezio-melottiAwaiting requested review from ezio-melottiezio-melotti is a code owner
@AA-TurnerAA-TurnerAwaiting requested review from AA-TurnerAA-Turner is a code owner
@webknjazwebknjazAwaiting requested review from webknjazwebknjaz is a code owner

Assignees

@hugovkhugovk

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@sandole@AA-Turner@picnixz@vstinner@sobolevn@JelleZijlstra@hugovk