Skip to content

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Feb 7, 2025

I can confirm that this now passes --strict-bytes check from mypy:

(.venv) ~/Desktop/cpython2  issue-129805 ✔                                                
» mypy --version
mypy 1.15.0 (compiled: yes)
                                                                                           
(.venv) ~/Desktop/cpython2  issue-129805 ✔                                                
» mypy --config-file=Tools/jit/mypy.ini --strict-bytes            
Success: no issues found in 6 source files

We can yet use [email protected] in CPython to verify it, because there are known regressions :(

@sobolevn sobolevn merged commit 422f8e9 into python:main Feb 17, 2025
70 checks passed
@sobolevn sobolevn added the needs backport to 3.13 bugs and security fixes label Feb 17, 2025
@miss-islington-app
Copy link

Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington-app
Copy link

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

cherry_picker 422f8e9e02e68d45aee3846751a003a70fca13b6 3.13

sobolevn added a commit to sobolevn/cpython that referenced this pull request Feb 17, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 17, 2025

GH-130216 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Feb 17, 2025
sobolevn added a commit that referenced this pull request Feb 17, 2025
brandtbucher added a commit to brandtbucher/cpython that referenced this pull request May 6, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 16, 2025
hugovk pushed a commit that referenced this pull request Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants