agmonks / nx-vue

Nx Vue Plugin
Other
3 stars 1 forks source link

Is there any way to make the logs optional? #6

Open leggomuhgreggo opened 1 year ago

leggomuhgreggo commented 1 year ago

It adds a lot of noise to the CI. Since the plugin runs for any Nx task, it shows up in unexpected places

The log statements are already abstracted to this log function (Nice!) https://github.com/agmonks/nx-vue/blob/2dd1e328406410b71d33ef13d342c666be4e3d29/libs/vue/src/processProjectGraph.ts#L18-L20

Cloud switch it to logger.debug https://nx.dev/packages/devkit/documents/nx_devkit#logger

Or condition it behind a DEBUG var

Would be happy to open a PR!

Thanks!

leggomuhgreggo commented 1 year ago

Tried changing it to logger.debug w/ patch-package and it seems to work agreeably

diff --git a/node_modules/nx-vue/src/processProjectGraph.js b/node_modules/nx-vue/src/processProjectGraph.js
index a006979..36e856f 100644
--- a/node_modules/nx-vue/src/processProjectGraph.js
+++ b/node_modules/nx-vue/src/processProjectGraph.js
@@ -10,7 +10,7 @@ const target_project_locator_1 = require("nx/src/plugins/js/project-graph/build-
 /* eslint-disable @typescript-eslint/no-var-requires */
 const path = require('path');
 function log(text) {
-    devkit_1.logger.info(/*'test.txt',*/ '[nx-vue]:' + text + '\n');
+    devkit_1.logger.debug(/*'test.txt',*/ '[nx-vue]:' + text + '\n');
 }
 function processProjectGraph(graph, context) {
     return tslib_1.__awaiter(this, void 0, void 0, function* () {