Skip to content

Conversation

jorenham
Copy link
Contributor

closes #19737
ref: #19737 (comment)


It's not the prettiest code, but since it will be removed once PEP 800 gets accepted (or rejected), I figured it would be best to keep it simple, so that we can easily revert it.

@jorenham
Copy link
Contributor Author

cc @JelleZijlstra

@jorenham jorenham changed the title stubtest: temporary --check-disjoint-bases flag [stubtest] temporary --check-disjoint-bases flag Aug 26, 2025
@jorenham
Copy link
Contributor Author

Yea, I can see how opt-out would be the path of least resistance here... I guess I'll go for the opt-in approach in the hope that PEP 800 won't be rejected.

@jorenham jorenham force-pushed the stubtest--check-disjoint-bases branch from 1b416a0 to 3c69535 Compare August 26, 2025 23:16
@jorenham jorenham changed the title [stubtest] temporary --check-disjoint-bases flag [stubtest] temporary --ignore-disjoint-bases flag Aug 26, 2025
@jorenham
Copy link
Contributor Author

I just verified on that the flag works by running this on the numtype numpy stubs

(sorry for noise btw, I'll shut up now)

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.

stubtest: don't enforce @disjoint_base by default until PEP 800 is accepted
2 participants