Skip to content
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

Show errors during static page generation #40

Merged
merged 1 commit into from
Sep 23, 2015

Conversation

fson
Copy link
Contributor

@fson fson commented Sep 22, 2015

Previously when errors were thrown during static page rendering, they
were swallowed by static-site-generator-webpack-plugin and the page was
rendered with blank content.

This commit adds a check to stats.hasErrors() and returns the error
string from stats to the callback. However, the stats object does not
expose the original errors, so we also print the stack trace to make
it possible to debug errors only thrown during server rendering.

Previously when errors were thrown during static page rendering, they
were swallowed by static-site-generator-webpack-plugin and the page was
rendered with blank content.

This commit adds a check to stats.hasErrors() and returns the error
string from stats to the callback. However, the stats object does not
expose the original errors, so we also print the stack trace to make
it possible to debug errors only thrown during server rendering.
KyleAMathews added a commit that referenced this pull request Sep 23, 2015
Show errors during static page generation
@KyleAMathews KyleAMathews merged commit 49fa931 into gatsbyjs:master Sep 23, 2015
@fson fson deleted the show-errors branch September 23, 2015 07:43
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
* fix auto fixable ESLint warnings

* use .jsx file extension

* move Header into it's own component

* use .js insteda of .jsx file extension

* format code using prettier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants