search

Gruntfuggly / todo-tree

Use ripgrep to find TODO tags and display the results in a tree view
Other
1.4k stars 135 forks source link

Please remove console logs from published extension code #724

Closed jayu closed 1 year ago

jayu commented 1 year ago

Hey, I have this extension installed and I've noticed it prints a ton of logs in extension development console

image

It makes it hard to develop other extensions with such a frequent logging. I think this line might be the reason

https://github.com/Gruntfuggly/todo-tree/blob/598a46be532b7d51814face262ff3ec42630664c/src/config.js#L203

Please remove logging if possible 😁

If you want to keep console.logs for development, you could perhaps use babel plugin like https://www.npmjs.com/package/babel-plugin-transform-remove-console, but it would require some additional setup since you are using ts-loader

Or setup custom logger function that would just output nothing in production env.

Gruntfuggly commented 1 year ago

Yes - sorry, that shouldn't be there. I also hate it when extensions pollute the logs. I'll remove it for the next version (won't be too long).

Gruntfuggly commented 1 year ago

Should be removed in the latest version (0.0.224)