drop hot calls for prod mode

[smelukov]

closes #1080

[codecov-io]

Codecov Report

Merging #1081 into master will increase coverage by 0.28%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##           master    #1081      +/-   ##
==========================================
+ Coverage    86.9%   87.18%   +0.28%     
==========================================
  Files          31       32       +1     
  Lines         840      874      +34     
  Branches      194      206      +12     
==========================================
+ Hits          730      762      +32     
- Misses         91       93       +2     
  Partials       19       19

Impacted Files	Coverage Δ
src/babel.prod.js	94.11% <94.11%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 696da9b...f7b922c. Read the comment docs.

[smelukov]

Now supports all import cases:

import { hot } from 'react-hot-loader'
import { hot as namedHot } from 'react-hot-loader'
import * as RHL from 'react-hot-loader'

[theKashey]

Amazing! Just amazing!

[smelukov]

Are there any comments for this? )

[theKashey]

Gonna handle this and another pending PRs on weekends.

[theKashey]

Released in 4.3.12

[smelukov]

Thanks :)

[edmorley]

Hi!

I don't see any change in the babel output when using default exports. Is this expected?

eg:

import React from 'react'
import { hot } from 'react-hot-loader'

const App = () => <div>Hello World!</div>

export default hot(module)(App)

[smelukov]

@edmorley Hi!
The output will change only in the production build (when NODE_ENV is production)

[edmorley]

Yup, this is with NODE_ENV=production, and webpack mode production.

[smelukov]

@edmorley thank you! I've investigated this and find out that hot-dropping may not work with babel 7 (no matter which kind of export do you use).
Fix in #1093