standardize the error message output? Some error you are using Console.Error.WriteLine. Some you are using Console.WriteLine, which is going to stdout. Does not have to be in this PR but right now, it is very confusing.
Originally posted by @jarupatj in #800 (comment)
UPDATE:
We will use logger in CLI as well to be consistent with the Engine code and also it will have the below advantages:
The logger gives to ability to define different levels of importance of the logged messages and the ability to use different sink for the output - the console, a file, etc.
Also it's easy to enable or disable only some type of message when using a logger - for example you don't want to see every debug message in production.
standardize the error message output? Some error you are using
Console.Error.WriteLine. Some you are usingConsole.WriteLine, which is going to stdout. Does not have to be in this PR but right now, it is very confusing.Originally posted by @jarupatj in #800 (comment)
UPDATE:
We will use logger in CLI as well to be consistent with the Engine code and also it will have the below advantages:
The logger gives to ability to define different levels of importance of the logged messages and the ability to use different sink for the output - the console, a file, etc.
Also it's easy to enable or disable only some type of message when using a logger - for example you don't want to see every debug message in production.