fix: avoid dynamic requires

[blimmer]

This PR is very similar to #113 but retains the lazy-load behavior per @evanw's comment.

This change would enable usage of esbuild and other single-file build processes.

[freben]

@DABH or @marikaner, is this something you think looks good?

We are hitting significant issues with the dynamic imports, and it seems that others are too. In our case the problems started when using mock-fs intests, which broke winston code.

[vitalii-boink]

is it possible to get it merged?
In my case it's breaking esbuild when bundling AWS CDK lambda's code

[vcfvct]

Can we pls merge this and publish with a new version so that we can bundle tools like esbuild without doing custom hacks.

Thanks!

[vikasgarghb]

Can this be merged as this is proving to be a blocker?

[vitalii-boink]

@blimmer is there anything else blocking you from having it merged?

[blimmer]

Not from my perspective - just need someone with permissions to merge and publish.

[vitalii-boink]

@blimmer do you know anyone with such permissions ? maybe we could tag someone

[blimmer]

Sadly, I do not. This is my first time contributing to this repo.

[vitalii-boink]

I see some people in this repository, maybe @indexzero, @mattberther, @pose , @chjj , @santigimeno , @mgibeau , @yurijmikhalevich , @ChrisAlderson might help to move to the next step? Thanks in advance!

[Magzter]

This would really help us, seems to be held up for a while. Changes look good.

[tgillus]

I just ran into this exact issue today when exploring using esbuild for a project at work. I'm looking forward to this being merged and released. Thank you.

[vitalii-boink]

@yurijmikhalevich are you able to merge this PR?

[empperi]

Any news on this? Why isn't this merged? It's an old PR proven to be good and it's a blocker for bunch of people?

[yurijmikhalevich]

@vitalii-boink, nope.

[vitalii-boink]

in this case maybe @indexzero could help us as creator of Winston ? so many libs depend on this and we cannot use them because of esbuild failure

[muldos]

+1 : just ran into this bug, and it seems to be the final that prevent my feather express app to run serverless

[empperi]

Last PR to this project was merged July 11th, 2020. Is this project still alive? Are there anyone left who can actually merge PRs?

[StoutyAlex]

This would fix all the issues I am facing with winston using eslint

[IceSentry]

For anyone that needs this fix, the actual code changes are really small, so you can use something like https://www.npmjs.com/package/patch-package to patch it manually

[yurijmikhalevich]

Now I can merge this :) Merged and released as part of v2.3.0. Thank you!