Mehdi Achour

Results 279 comments of Mehdi Achour
trafficstars

@ikarus-akhil you could provide a PR to add support for both files extensions in order to speed up things. Check https://github.com/remix-run/remix/pull/3985 for an example on how to do that.

Seems to have been fixed upstream in esbuild https://github.com/evanw/esbuild/issues/2534 Fix will probably land in esbuild 0.15.8

@gdonoso94 as you can see, there was a PR that went stale and got closed. The solution is to open a new PR to expose the esbuild `externals`option.

Definitely, but until then, the issue should be properly documented for our users. Thank you again for your contribution, already using it! 🙏

Hi @joelazar, thank you for your submission (and sorry for the delay!) Can you target `dev` instead of `main` as per our [contributing guide](https://github.com/remix-run/remix/blob/main/docs/pages/contributing.md#making-a-pull-request) requirement?

@brophdawg11 is there a decision doc for this work? Would help me follow along

@nkrmr do you mean 1.7.4? Bug was reported on 1.7.2 and the fix haven't been merged yet, see #4301

@IgnisDa could you provide a reproduction somewhere? Might be a problem with our web-fetch package

I successfully reproduced the issue with NodeFileOnDisk 👍 However, I'm not sure we will be able to fix errors from Cloudflare pages & workers, as they seem to emanate from...

The error message seems quite self-explanatory: top level await are not available for node14. Can you try with node16 or maybe node14.14 (see https://github.com/evanw/esbuild/issues/253#issuecomment-1042859888)?