Page MenuHomePhabricator

[native] Get rid of "waow cookie" message
ClosedPublic

Authored by ashoat on Jan 7 2023, 1:39 PM.
Tags
None
Referenced Files
F3529525: D6194.diff
Tue, Dec 24, 7:27 PM
Unknown Object (File)
Mon, Dec 16, 8:21 PM
Unknown Object (File)
Thu, Nov 28, 10:38 PM
Unknown Object (File)
Nov 24 2024, 11:56 AM
Unknown Object (File)
Nov 24 2024, 11:56 AM
Unknown Object (File)
Nov 24 2024, 11:56 AM
Unknown Object (File)
Nov 11 2024, 3:35 PM
Unknown Object (File)
Nov 4 2024, 6:02 AM
Subscribers
None

Details

Summary

After the React Native 0.70 upgrade, I noticed a statement saying something like "wow you found the file, here's a cookie ๐Ÿช". I couldn't find that anywhere on GitHub or via git grep, so I ran grep -R ๐Ÿช . in the root in the repo and found what was going on.

The source was the @favware/skip-dependency package. I found this issue on their GitHub, which revealed that the line got printed when the dependency being replaced by @favware/skip-dependency was included. It's there as a sort of canary to indicate that the replaced package is still being imported, and to warn of any possible issues that may result.

In our case, the import is occurring in @react-navigation/devtools, but since we're not using useFlipper from that package, nothing is done with the import, so we are safe.

To turn off the warning, I updated the package to a version that no longer printed the error, as advised in the linked GitHub issue above.

Test Plan

Confirm the cookie message was no longer getting printed in the React Native logs

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable