Skip to content

Conversation

@belav
Copy link
Owner

@belav belav commented Mar 4, 2024

This includes the changes needed to have csharpier return errors using the server option.
And also fixes an issue with the stdout version causing hangs if it through an exception, it would never write out the "finished" character.

@belav belav force-pushed the server-error-handling branch 3 times, most recently from 800e831 to 8135d1f Compare March 4, 2024 02:00
@belav belav force-pushed the server-error-handling branch from 8135d1f to a44ef14 Compare March 9, 2024 00:23
belav added a commit that referenced this pull request Mar 9, 2024
@belav belav added this to the 0.28.0 milestone Mar 9, 2024
@belav belav marked this pull request as ready for review March 9, 2024 00:26
@belav belav changed the title Returning errors from csharpier cli. Make sure we don't hang the old stdout version when cli fails Returning errors + status from csharpier server. Make sure we don't hang the old stdout version when cli fails Mar 9, 2024
@belav belav merged commit 1ee22ea into main Apr 7, 2024
@belav belav deleted the server-error-handling branch April 7, 2024 21:29
belav added a commit that referenced this pull request Apr 12, 2024
belav added a commit that referenced this pull request Apr 12, 2024
* Rider changes needed to support #1191

* Update CSharpierProcessServer.java

* Always use server when version >= 28

* update readme

* Deal with deprecated warning

closes #1224

* Some cleanup + logging version

* bump version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants