strager
strager
> The thing is it's a small variation on another error, the used_variable class already has a 'kind' field, why not pass that to the error formatter instead of having...
Please keep this PR a draft until the feature is complete.
> @strager What you feel? is this on the right track? Unifying the plugin settings is a good idea!
> i recall you mentioned on a past code review a "what if quick-lint-js was installed with npm problem", Yup. > i infered that meant "look for quick-lint-js in node_modules",...
> Also, I'm still unsure about their need, i've written them because i'm unsure if emacs generated files are forward and backward compatible, so i left that to the host...
> [316e270](https://github.com/quick-lint/quick-lint-js/commit/316e2704d06f40d0183e1ffafd30fd0f464d2c4f) fixes #395 by adding a wrapper command which marks the `default-directory` as the root folder, unfortunately i think we can't transparently fix this, my first attempt was to...
@wgrr Are you waiting for feedback from me? The PR is in a draft state still.
ping @wgrr
This might help understanding: https://dev.to/rkirsling/tales-from-ecma-s-crypt-annex-b-3-3-56go
It looks like Node.js supports it: https://nodejs.org/dist/latest-v18.x/docs/api/esm.html#data-imports