Windows does not understand "export"

orangecoding / fredy

:heart: Fredy - [F]ind [R]eal [E]states [D]amn Eas[y] - Fredy will constantly search for new listings on sites like Immoscout or Immowelt and send new results to you, so that you can focus on more important things in life ;)

http://www.orange-coding.net

MIT License

231 stars 58 forks source link

Windows does not understand "export" #51

Closed sven-simonsen closed 2 years ago

sven-simonsen commented 2 years ago

Describe the bug When running yarn run prod on a windows machine the command export BUILD_DEV='false' in package.json fails with 'export' is not recognized as an internal or external command.

Expected behavior Replacing all instances of export with set if running on a windows machine fixes the issue. I don't know how to do that in a dynamic way though.

orangecoding commented 2 years ago

I was planning on using this: https://www.npmjs.com/package/run-script-os

Would you mind checking it and send a pr if it works coz I don't have any windows os.

sven-simonsen commented 2 years ago

Added a PR from a fork. Dunno if that was the correct way to do it, if not just copy the change to a better structured branch.