Remove "react-🔥-dom patch detected. You may use all the features." console messages

[stnwk]

Hello :)

It would be nice if we could get rid of this message in the console too:

React-Hot-Loader: react-🔥-dom patch detected. You may use all the features.

It's possible to silence all other console logs, but not this one :/

See

react-hot-loader/src/reactHotLoader.js

Line 118 in f15b108

console.info(

What do you think?

[theKashey]

Probably it's better to display a message like this when hot patch is not detected, as long as we rely on it.

[a7madgamal]

how to fix it then?

[theKashey]

The current version would display this message only if hot-patch is hot present.

React-Hot-Loader: react-🔥-dom patch is not detected. React 16.6+ features may not work

Cos, you know, 16.6+ features really might not work, especially hooks. React-Hot-Loader is not very reliable thing and might ruin all DX with eeeeaaaasssyyyy 😭. We need some support from React side but have to inject it by ourselves.

[mohsinulhaq]

38b63cd
This commit said it's supporting 16.8.4, why do we still say it doesn't support 16.6+?
Are there any known issues with 16.6+?

[theKashey]

@mohsinulhaq - you just linked a common in webpack loader, which is a "react-🔥-dom patch".
There are a few things RHL cannot do from a "user-space" properly. For example, hooks are breaking some internal stuff, and we are injecting additional API into react-dom to actually make them work without surprices.

So - long story short - all the tests are passing with and without hot patch. Everything is about reducing WTF per minute. RHL is a hacky tool by nature - we are hacking things.

[mohsinulhaq]

Ok understood, so what do you think would be a proper solution? Does something need to change in react-dom officially?

[theKashey]

reactjs/rfcs#74 - but it's a bit stale.