Webpack compile twice when js file changed about prettier-loader

[garyxuehong]

Thank you for spending time investigate it. I am actually totally happy to have it compiled twice. Seeing multiple green 'success' displayed on screen makes me feel that my code is rock solid :)

[iamolegga]

Hi, sorry for so late, it's done in watch-mode-fix branch, but I want to be sure with this, and I would like to take just a couple of days for more testing before merge it to master

[iamolegga]

Sorry for delay, I could not find a lot of time for solving this problem before.

But I have time on next week, so I think somehow this will be done.

[iamolegga]

Hi, sorry for late, found this issue just now :) I'll try to do something with it

[iamolegga]

I'll try to figure out the problem:
user save changes to file -> here comes first compilation
if loader sees that file should be formatted, it overwrites the file -> here comes next compilation

Seems like it's not possible to solve this problem by only loader, according to loader api you can not prevent already started compilation.

But, I think this is where we can create plugin, that prevents second compilation.
And this is what you will add only to your dev config of webpack.

Need more time to solve this problem.

[iamolegga]

Webpack team add new documentation on loaders api, so it seems like .pitch should help to solve this problen

I'll take a look in next few days

[enzoferey]

How the research is going ? Really looking forward it :)

Amazing loader btw, saved me a lot of headaches with pre commit setup inside Docker 👍

[enzoferey]

Hi ! How is it going ? Looking forward it 😄

[Deadly0]

@iamolegga hi, will you merge the fix? Maybe publish beta version with the fix to npm?

[iamolegga]

@Deadly0 I tested this, but had some issues with webpack 4, so that's why I didn't merge it yet. Now I'm on vacation, and I have not access to npm, so more time is required

[sola92]

@iamolegga any luck with this?

[dsktschy]

Any solution for this?

[iamolegga]

Hi everyone, first of all, I'm so sorry for making you waiting so much time for solving this.

Last 2-3 days I was working only on this problem trying to solve this old bug. I've read all the webpack API docs, I've tried different ways, but no luck. With current webpack API it seems like it's not solvable.

More over webpack maintainers saying that loaders should not do things like this loader does (webpack/webpack#9763) 🤷‍♂️

I've looked at other similat projects, but no one has solution.

So I'm going to add this as knowing issue in docs, and closing it for now.

Sorry

[madnilo]

Anyone found a good alternative to this?
I am using the package in my project, and wondering what else is out there.