-
Notifications
You must be signed in to change notification settings - Fork 51
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
"watch""check" return nothing #199
Comments
update... https://github.com/pschanely/CrossHair/tree/main/crosshair/examples/PEP316/bugs_detected |
If there is way to check or debug why it returns nothing, let me know how . |
Sorry - I'm not sure whether I understand; did you already resolve your issue, or no? A few quick notes from your screenshot:
At any rate, if you haven't already solved the problems, maybe paste the code directly here so that I can reproduce; and/or try it at crosshair-web.org and make a Gist to send me. (this function that you're attempting reports an error, for example)
CrossHair should better features for this - sorry.
|
Thanks for your kind. what I tried on playground then, I tried same code on my local env, target file path crosshair seems not to check
|
OH! You caught an interesting kind of bug. Essentially, crosshair needs to import your code a module under some name; in your case that module will be named "crosshair" which conflicts with crosshair itself. (the same problem would happen for files named like standard library modules, e.g. "sys.py", "typing.py", etc) For now, you can either rename "crosshair.py" to something else, or create an empty Assuming my diagnosis is correct (please confirm), I'd like to repurpose this issue to ensure crosshair reports an error in cases like these. |
Solved! Maybe this your answer will solve #198 |
Excuse me.
Let me know why
crosshair check
return nothing?Example codes are from:
https://crosshair.readthedocs.io/en/latest/kinds_of_contracts.html
https://crosshair.readthedocs.io/en/latest/contracts.html
these codes should return breaking contract, I think,,,
The text was updated successfully, but these errors were encountered: