-
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
chore(gatsby-source-contentful): Fix stack overflow cycles #20674
Conversation
I am also experiencing issues with the plugin exceeding the stack size. I'd love to check if your pull request solves these issues. Yet I am struggling to link the pr with my package.json due to the gatsby repo being a monorepo. Do you know of an easy way to try out this pr? |
You could try to patch node_modules manually. I don't expect this to take too long to get merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I don't know much about this code but I don't see any problems with it. I added some nits and asked for a few comments ^^
c5a6308
to
a55eb2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels sane. The only thing I am left wanting for is you describe a spec where if the id
is a number, then you prefix it with c
. However I'm not seeing a snapshot for this case. Would love to see one. But the code looks solid!
#resolved :) |
This fixes the repro reported in #11364