Lam Wei Li

Results 136 comments of Lam Wei Li

> _Originally posted by @peteriman in https://github.com/log4js-node/log4js-node/issues/1171#issuecomment-1023423008_ > I changed your codes a little so I can get some initial findings real quick. > > ```javascript > const log4js =...

There are some challenges with this, as the `pattern` is not intended to be used like this. It was intended to be a suffix of the filename, adhering to `date-format`...

@CQNightBreeze Would need the full code for the `log4js.configure()`, include the `appenders` section.

The current red layout (31) works for me though. The new one (91) seems dimmer.

Would you mind providing some details of what is expected?

`--enable-source-maps` is no longer experimental from v14.18.0 (LTS) onwards. https://nodejs.org/dist/latest/docs/api/cli.html#--enable-source-maps We should consider [source-map-support](https://www.npmjs.com/package/source-map-support) for older Node.js (>=8.0) if anyone has the time.

ESM should be able to load CommonJS but not vice versa as far as I recall.

@nicojs, is line 27 the correct line to change? https://github.com/log4js-node/log4js-node/blob/2f29d0a0f5c2fcb8c5f58d9d6aeec1712ee104b7/lib/appenders/index.js#L24-L32 Would you be kind enough to do a PR? 😊

I tried to change the codes in `appenders/index.js`, but to no avail. ```diff - const tryLoading = (modulePath, config) => { + const tryLoading = async (modulePath, config) => {...