Skip to content

Fix instancecheck, subclasscheck of UnionType#3984

Merged
youknowone merged 2 commits intoRustPython:mainfrom
Karatuss:fix-issubclass
Jul 31, 2022
Merged

Fix instancecheck, subclasscheck of UnionType#3984
youknowone merged 2 commits intoRustPython:mainfrom
Karatuss:fix-issubclass

Conversation

@Karatuss
Copy link
Copy Markdown
Contributor

Resolve test_types::UnionTests::test_instancecheck, test_bad_instancecheck,test_bad_subclasscheck

@Karatuss Karatuss changed the title Fix instancecheck, subclasscheck of UnionType Fix instancecheck, subclasscheck of UnionType Jul 29, 2022
@youknowone
Copy link
Copy Markdown
Member

@Snowapril could you review this PR?

@youknowone youknowone added the z-ca-2022 Tag to track contrubution-academy 2022 label Jul 29, 2022
Comment thread vm/src/builtins/union.rs Outdated
@Karatuss
Copy link
Copy Markdown
Contributor Author

I made test_types.py update by referring to CPython 3.10's one for more precise checks.

@youknowone
Copy link
Copy Markdown
Member

youknowone commented Jul 31, 2022

The changes looks great, thank you! Just one more request. Could you rebase this PR to 2 commits - one for test changes and the other one for non-test code changes?
When I'd going to squash all commits to single commit, I usually can do it myself with github UI, but not when I wants more than single commits.

Copy link
Copy Markdown
Member

@youknowone youknowone left a comment

Choose a reason for hiding this comment

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

Great! Thank you!

@youknowone youknowone merged commit ac48968 into RustPython:main Jul 31, 2022
@Karatuss Karatuss deleted the fix-issubclass branch August 1, 2022 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

z-ca-2022 Tag to track contrubution-academy 2022

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants