styleguidist/react-styleguidist

env is hardcoded to "production" which causes React v17 to fail building properly

Open

#1,721 建立於 2020年11月30日

在 GitHub 查看
 (7 留言) (0 反應) (0 負責人)TypeScript (1,416 fork)github user discovery
buggood first issuehelp wanted

倉庫指標

Star
 (11,091 star)
PR 合併指標
 (PR 指標待抓取)

描述

Current behavior

I think there's a mismatch of environment variables being read by babel (JSX config) and Styleguidist, causing some issues deeper down.

(I've noticed this was raised before but was closed)

So the issue is. My webpack config has mode set to "production" (because mode comes from env) and anything else that uses NODE_ENV will see developement.

I ended up with this error: ReferenceError: _jsx is not defined.

I don't know why its hardcoded to production regardless of what you set, but it seems to be the cause of this bug.

Potential Fix https://github.com/styleguidist/react-styleguidist/blob/master/src/scripts/make-webpack-config.ts#L29 should be

env = process.env.NODE_ENV || env

https://github.com/styleguidist/react-styleguidist/blob/master/src/scripts/build.ts#L9 should also default to 'development' instead of 'production'

貢獻者指南