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?
josevalim commented
Checking the version is a good idea. Can you please send a PR?
josevalim commented
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.
josevalim commented
@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.
josevalim commented
Thank you for double checking and once again, sorry for the double effort. :)