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

Debug.WriteLine in place of Console.WriteLine #59

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

remigarcia
Copy link

Hello,

I'm using this library in a project which is communicating via CGI with a nodejs project. The latter takes the console output of the C# project in a json format. My problem is that when the iRacing yaml fail to parse via the YamlParser, the error is sent to the standard output via the Console.WriteLine which pollute the output and any client connected to the app fails to parse as it's not JSON.

As it is an error message and a stack trace, I propose to write them in the Debug trace listeners instead, that way we can still see them in a debug env.

Hello,

I'm using this library in a project which is communicating via CGI with a nodejs project. The latter takes the console output of the C# project in a json format. My problem is that when the iRacing yaml fail to parse via the YamlParser, the error is sent to the standard output via the `Console.WriteLine` which pollute the output and any client connected to the app fails to parse as it's not JSON.

As it is an error message and a stack trace, I propose to write them in the Debug trace listeners instead, that way we can still see them in a debug env.
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.

1 participant