-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do NOT use this! #12
Comments
@davwheat I have installed the plugin, and then seen your comment. These are the current dependencies in my package.json: |
@luba-media You just need to remove this plugin. Running To be extra sure, use the Gatsby Webpack Bundle Analyzed plugin and build your site locally -- you should be able to see all your deps there and check there's no duplicate React. By the way, if you want to cut down that bundle size more, check our the Gatsby Preact plugin. Preact is a smaller version of React and Preact can be an almost drop-in replacement unless you're doing VERY complex things with React. |
@isamrish moving react to the peer-dependancy should solve this problem. |
Do not use this plugin!
1. React is a dependency
This plugin has react as a dependency: it should not be. If you use this plugin, you'll end up with 2 different React packages in your bundle, increasing its size significantly.
2. You don't even need it!
Want ads? Copy
.cache/default-html.js
to yoursrc
folder, renaming it tohtml.js
. Add your AdSense script tag to the head of that file. This replicates this plugin exactly.The text was updated successfully, but these errors were encountered: