-
Notifications
You must be signed in to change notification settings - Fork 24
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
Errors should be written to stderr not stdout #611
Comments
It looks like core_d doesn't support this currently (see mantoni/core_d.js#19). It makes sense though. I'll try to find some time to send a PR to core_d. |
@fsouza is there a reason this behavior is still occuring? Do I need a new release of prettierd? |
Hmm interesting, that release should include the updated core_d, I wonder where the issue is. I'll check what's up. |
@fsouza have you figured out whats going on here? is there anything anyone can help with? |
I haven't had a chance to look into it yet. My plan was to try to repro and trace down whether some errors are being written via the wrong function in core_d or something like that. |
I just took a look into this, it's actually a bit trickier to fix, as with core_d you only know whether something failed or not at the end of the stream. We'd need to support the concept of stream when reading data from the socket. I started an issue in core_d where we can discuss it: mantoni/core_d.js#28 |
@fsouza in the linked core_d issue (mantoni/core_d.js#28), the maintainer replied. does the stuff in 6.1.0 solve the issue and prettierd just needs to upgrade core_d again? |
@jeffshaver that will help debugging, but won't stop errors from being sent to the editor. I'll push a new release of prettierd with that feature though. |
0.25.3 - it's still actual. 😞 And one more issue: it's impossible to switch off colors. This breaks other things: stevearc/conform.nvim#486 |
Let's see if we can get this resolved. Is there any way I can help @fsouza ? |
Any updates? |
Write errors to stdout. prettier does the same.
The text was updated successfully, but these errors were encountered: