ryyppy / rescript-nextjs-template

Opinionated Boilerplate for NextJS, Tailwind and ReScript
https://nextjs-default.ryyppy.now.sh
Apache License 2.0
407 stars 33 forks source link

Need for FS shim #43

Closed ghost closed 3 years ago

ghost commented 3 years ago

In the rescript-lang site, the fs api appears to only be used by getStaticProps implementations. I never understood why the empty shim is needed if nextjs takes care of compiling and extracting getStaticProps for the node environment. Consider adding more documentation to next.config.js.

ryyppy commented 3 years ago

There are some instances where code is getting pulled into e.g. the blog. We recently refactored a lot of code because we switched to a full mjs setup, so I am not even sure if this is still relevant anymore. Will need to try.