-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: --beartype-skip-packages
#9
Conversation
Brilliant! I rejoice to know you. This is me awkwardly saying...
srsly. I meant to give you a detailed writeup integrating this Sadly, I have no money. I can only thank you. Please accept this ceremonial Canadian turkey: 🦃 |
I'll probably have to wait until this arg has been released on PyPI. Will it be part of 0.19? |
Testing! Forgot about that. Testing is... cumbersome. Here's what @beartype's test suite does:
# Assert that a PEP 526-compliant assignment statement assigning an object
# violating the type hint annotating that statement raises *NO* exception.
this_should_fail_but_doesnt: str = b'Totally not a string, but nobody cares.'
assert isinstance(this_should_fail_but_doesnt, bytes)
Thanks again being so awesome, @tusharsadhwani. Kali herself surely shines upon you. |
Totally. 0.19.0 is the plan. Alternately, users could be forced to install the live @beartype if they really want this. I am fine with forcing users to do stuff they don't want to do. Let them eat Totally up to you. Thanks so much yet again. I get the sinking feeling I'd better start hacking on 0.19 a lot harder. 😄 |
You already hack at an incredible pace. There's no need to worry. |
oh right, i can ship this now. |
This should work, all that's left is to actually verify if it does or not.