-
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
Update graphviz dependency #268
Conversation
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.
LGTM
This appears to have fixed the problem although the build has a number of errors about event emitter @JosephusPaye I don't think it has stopped the build but not sure if it's a major problem |
That error you're seeing occurs when you have an event emitter which has more than 10 distinct listeners for the same event. Although the warning is not 100% accurate, when it shows up it usually means something weird is happening in the code (like registering event listeners in a loop). We can get more information about where the issue is happening by running this... node --trace-warnings node_modules/.bin/gatsby build ... which produces this output:
Showing that the issue is in Gatsby itself (none of the stack entries are our code). Perhaps there's an updated version of Gatsby where this is fixed. A quick search shows Gatsby's had similar issues before: |
Ah thanks, I wasn't sure what path I was supposed to put after the |
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.
LGTM!
No description provided.