Skip to content
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

Using mypy to verify check_str_format_call? #82

Open
kornicameister opened this issue Jan 5, 2021 · 2 comments
Open

Using mypy to verify check_str_format_call? #82

kornicameister opened this issue Jan 5, 2021 · 2 comments
Labels
question Further information is requested

Comments

@kornicameister
Copy link
Owner

@ThibaultLemaire Recently I found mypy.checkexpr.ExpressionChecker.check_str_format_call method.
I think that is basically what you did as part of #43, right? If that's exactly same thing would you mind replacing your code with the one referenced.

@kornicameister kornicameister added the question Further information is requested label Jan 5, 2021
@ThibaultLemaire
Copy link
Contributor

I think that is basically what you did as part of #43, right?

Correct. visit_call_expr calls visit_call_expr_inner which in turn calls check_str_format_call. I even found some of my old breakpoints in there which means I must have been aware of its existence.

I believe I had a good reason not to call it directly, but I can't recall and I don't see anything important in visit_call_expr_inner now. Maybe visit_call_expr was too big and I was too happy that my code worked to dig any deeper 🤷‍♂️

Well spotted, feel free to change it if you want 👍

@kornicameister
Copy link
Owner Author

If I get some time, I will give it a shot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants