phoenixframework/esbuild

Should `esbuild.install --if-missing` check also if outdated

Closed this issue · 6 comments

cw789 commented

Or should I add a --update-only flag?

What do you thing?

Checking the version is a good idea. Can you please send a PR?

I had some free time and I will push it. :)

cw789 commented

Ohh, my PR is up. By checking Esbuild.configured_version() we will unexpectedly update for users if the latest_known version within esbuild is updated and no version is configured.

@cw789 I think that behaviour is the desired one, after all, esbuild is under a lock file too. Otherwise people on the same esbuild version will get different outputs depending if they ran mix esbuild.install in a previous version, no?

cw789 commented

You're right. Got my head around. Thanks for learning.

Thank you for double checking and once again, sorry for the double effort. :)