search

imjuni / ctix

CLI to generate barrel file for webpack, rollup entrypoint
MIT License
82 stars 22 forks source link

CtixReasoner outputs empty strings to stderr when there are no true errors #79

Closed jR1P621 closed 1 year ago

jR1P621 commented 1 year ago

The CtixReasoner class arbitrarily calls this.#streamWrite(''); in several places. This causes false positives when ctix is called from tools that rely on stderr to determine error state.

imjuni commented 1 year ago

Hi @jR1P621, Thank you for your feedback!

Would you create reproducable repo?

jR1P621 commented 1 year ago

Here a repo using rushstack:

https://github.com/jR1P621/rushstackctix

imjuni commented 1 year ago

CtixReasoner default option is to use stderr, but this can be changed using CLI options. The reasonerStream and progressStream options can be used to change the stream to output messages to.

imjuni commented 1 year ago

@jR1P621 Hi,

I've working on ctix 2.x. In the 2.x version, Reasoner will not disply error messages multiple times and will not call console.error multiple times to print \n. Thank you for your feedback in improving ctix.

imjuni commented 1 year ago

fixed in v2.0.0