Improve component type check in getComponentKey.

[bmeurer]

The sequence

component && typeof component === 'object'

checks whether component is any JavaScript object except document.all.
Since document.all cannot occur here, this can be replaced with the
usual

typeof component === 'object' && component !== null

sequence, which yields true for all JavaScript objects and is well
optimized by all JavaScript engines.

[gaearon]

Looks good to me, thanks. Let’s wait for CI.

[bmeurer]

That being said: it obviously filters out callable objects still.

[bvaughn]

Looks like you need to run yarn prettier to format the code. Since the line's longer now, Prettier wants to split it:

if (
  typeof component === "object" &&
  component !== null &&
  component.key != null
) {

[bmeurer]

Done. Although it didn't split it into three lines.

[gaearon]

Likely because we haven’t updated to the last version yet (we will soon).

[bmeurer]

Anything else missing?

[bvaughn]

Oh, my bad. My local prettier did but I may have been using a diff version. :)

No, looks good. Thanks for following up!

[bmeurer]

Thanks!

[sophiebits]

Thanks @bmeurer!