New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: update esbuild to 0.16 #288
Conversation
See https://github.com/evanw/esbuild/releases/tag/v0.16.0 There is one breaking change with the legalComments default which changed to `none` and I just updated docs and test for this assuming you want to follow esbuild's default (I would).
I prefer to follow the default as well, but we'll need to avoid the breaking change to release this as a minor. |
Why not bump to 3.0.0? Want to include any other breaking changes in such a version? |
Ah, I see #275. Well to me personally, there is nothing critical in esbuild 0.16, so I'm happy to wait a bit longer until this is merged/released. |
Any update on when this can be merged in? I would like to test esbuild-loader but am unable to. There is a esbuild defect that was recently fixed and I can't test out esbuild-loader until it is in as a third party library I am using exposes this defect. |
@@ -338,7 +338,7 @@ Minify JS using equivalent but shorter syntax. | |||
#### legalComments | |||
Type: `'none' | 'inline' | 'eof'` | |||
|
|||
Default: `'inline'` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was reverted in https://github.com/evanw/esbuild/releases/tag/v0.16.10
See https://github.com/evanw/esbuild/releases/tag/v0.16.0
There is one breaking change with the legalComments default which changed to
none
and I just updated docs and test for this assuming you want to follow esbuild's default (I would).