-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[gatsby-source-filesystem] Deprecation warning #18433
Comments
Here is the documentation for this deprecation. I believe a fix will involve finding the offending dependency and updating it to a later version (if the newer version of that package has switched to the proposed methods in the doc). In the uncommon event that even the dependency has not yet fixed this deprecation notice, things might be murky, but I think we can handle it. This warning comes from something using the I'm assigning myself to this issue, I hope that's cool with everyone. |
Found the same issue on My guess is that
Either way, I will try |
Strange. I actually cannot reproduce this warning message with the Using Contentful Gatsby Example. @rayriffy can you update your packages and check if the warning still persists? |
This warning is still being present in
|
Ok, I've managed to identify the conditions under which this can be reproduced. This warning occurs only when This option is not set in the Gatsby provided example, so it does not show this warning there. The warning will also not show if there is nothing new to download from contentful in subsequent builds. So the problem does exist in the current version of the plugin. I still think |
Turns out the culprit was Discovering that was easy by tracing the deprecation warning's stack by running : node --trace-deprecation node_modules/gatsby/dist/bin/gatsby build The stack was as following :
I'm going to spend some time making sure the tests are passing before making a pull request. |
See #18857 (comment) - we are blocked with getting rid of deprecation notice until we can safely bump one of dependency. |
Don't worry, I can wait but I think it will be able to safely remove support for Node 8 now since it's not an active LTS |
Hiya! This issue has gone quiet. Spooky quiet. 👻 We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing! Thanks for being a part of the Gatsby community! 💪💜 |
Hey again! It’s been 30 days since anything happened on this issue, so our friendly neighborhood robot (that’s me!) is going to close it. Please keep in mind that I’m only a robot, so if I’ve closed this issue in error, I’m As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing! Thanks again for being part of the Gatsby community! |
Getting the same on Node 13 when using the |
Getting the same thing:
|
Is there any chance to get this solved quickly? Issue is hitting me on Gatsby Cloud and it's been a couple of months now that it was decided to wait for other dependencies. Apparently the same issue is present in |
Same issue with Node 13 |
Followed up with other members in the org, and this now seems to be in the process of happening. Once support for Node 8 is dropped, my PR above will be merged and this should be fixed. |
hi, |
I do believe the current release has my fix merged (not a full 100% sure), so if it is the same issue, you might want to update the version of gatsby and plugins in your project. If that doesn't help, I recommend running the same trace command I've used above to find out the cause of the DeprecationWarning and opening a new issue perhaps. |
Description
I got the deprecation warning from this plugin for Node 12 environment
Steps to reproduce
Just try to use
gatsby-source-contentful
Environment
The text was updated successfully, but these errors were encountered: