-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
rewriter: don't output target info to stderr #12255
rewriter: don't output target info to stderr #12255
Conversation
Make sure to fix the test failures too |
c8c8049
to
6325551
Compare
6325551
to
6c398ba
Compare
Any reason you closed this? |
It's been more than a year, what's the point? |
@benoit-pierre. Yeah, we really struggle with stuff falling through the cracks. This looks fine to me, actually. |
Unfortunately (and even the core maintainers have to do this too), we just sometimes have to ping the MR or matrix and ask for review of things. I'll merge this as long as CI is green when it's done. |
@benoit-pierre A rebase may be needed to get the linters happy |
Send the info to stdout, where it belongs.
6c398ba
to
8b6122c
Compare
Master is failing the lint job too, I guess due to the newly released 3.3.0 version of pylint. |
Fix for pylint: #13695. |
that's merged, thanks for doing that. |
Send the info to stdout, where it belongs.