aren't propTypes removed in production anyway? about babel-plugin-flow-react-proptypes HOT 7 CLOSED

Hmm... we might want to add an option for adding a NODE_ENV check so minifiers can strip it for production.

from babel-plugin-flow-react-proptypes.

yea, but aren't the propTypes removed in production automatically??

from babel-plugin-flow-react-proptypes.

Nope. React doesn't check them in production, but they're still in the code.

from babel-plugin-flow-react-proptypes.

Hmm... we might want to add an option for adding a NODE_ENV check so minifiers can strip it for production.

@brigand Was this ever implemented? AFAIK consumers of a library won't be able to run babel plugins like babel-plugin-transform-react-remove-prop-types on their imports unless they get messy with their node_modules so the only chance they have of stripping out prop types would be if the library had their prop types under NODE_ENV flags.

from babel-plugin-flow-react-proptypes.

Yeah, we could add this, and since require('prop-types') is inline, it'll get hit by dead code elimination. I'll be pretty busy the next few days, but I'll add a reminder to do this if no one gets to it first.

from babel-plugin-flow-react-proptypes.

Cool! I don't mind helping out if you need some :).

from babel-plugin-flow-react-proptypes.

@sohkai I'd love some help!

from babel-plugin-flow-react-proptypes.