-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Fix abc error messages #5140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix abc error messages #5140
Conversation
oh, |
The test_dataclasses is now a test folder instead of a single test file, and in what way should it be updated? |
We have 2 choices.
|
The test files in the folder seems to be unrelated with the test_dataclasses.py though |
I checked cpython repository. |
Oh I was checking out the 3.12 branch, why is my url not working :( |
Got it. I think the test is moved to https://github.com/python/cpython/blob/3.12/Lib/test/test_dataclasses/__init__.py If you run test through |
Oh, but we are still following the main branch right? Then I will update the test file and ignore the folder |
I prefer to follow recent distribution's tag. I'd like recommend v3.12.1 tag at this time. https://github.com/python/cpython/tree/v3.12.1 |
I found that test_dataclasses needs support from dataclasses.py, which the latter would went into the |
Marking test_dataclasses expectedFailure seems reasonable in that case. |
I made this temporary change here: https://github.com/RustPython/RustPython/pull/5140/files#diff-f0436842533448f8f02a19081d7c9cc8372b6685756caf32c83dfecebbee68e7R3679 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Fix #5137