rspack icon indicating copy to clipboard operation
rspack copied to clipboard

Published 20 hours ago • verified publisher icon web-infra-dev

feat(parser): add new javascript parser options

Open fi3ework opened this issue 1 year ago • 1 comments

Summary

Fix https://github.com/web-infra-dev/rslib/issues/204 and more.

We'll try to merge this back into webpack after stabilized.

Checklist

  • [x] Tests updated (or not required).
  • [ ] Documentation updated (or not required).

fi3ework avatar Sep 20 '24 10:09 fi3ework

Deploy Preview for rspack canceled.

Name Link
Latest commit f8891479f78622404bb4030cfd44bbff1c4d72b1
Latest deploy log https://app.netlify.com/sites/rspack/deploys/67065673f6622b00089d0bff

netlify[bot] avatar Sep 20 '24 10:09 netlify[bot]

I am ok with the current implementation, but we should keep it as experiment for now, or at least hide it from docs,

Sure. Any why to make it as experimental now? Or just using name like experimenalRequireResolve or rspackRequireResolve. I think adding an experimental warning in jsdoc is enough.

fi3ework avatar Oct 09 '24 09:10 fi3ework

An experimental warning in jsdoc and hide these options from rspack docs looks good to me

ahabhgk avatar Oct 09 '24 09:10 ahabhgk