search

solidjs / solid-refresh

MIT License
86 stars 18 forks source link

Update README.md - extended Parcel section #38

Closed mustevenplay closed 1 year ago

mustevenplay commented 1 year ago

Extended Parcel section with indications for its configuration file, in order to make the plugin work.

lxsmnsyc commented 1 year ago

Thanks for submitting this! The problem is not just with Parcel pulling in the production build, but Solid Refresh relies on two things that the development build has: DEV and devComp.

The former is used for detecting and warning if the user isn't using the dev build. Without DEV, files would just decline updates (default behavior of modules). The latter is used for refreshing components (which is crucial). Without those two, Solid Refresh is basically broken.

For now, this is the reason why Parcel is dropped (temporarily) but since conditional exports is now coming, we may consider re-adding it again. I'll leave this PR unmerged for now.

mustevenplay commented 1 year ago

Oh okay! I'll correct this again in the meanwhile.