-
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
docs: Update bootstrap sequence #19775
docs: Update bootstrap sequence #19775
Conversation
I've run the job twice on CircleCI and got two different errors, I'm not sure what the problem is 🤔. Any help, please? |
Hey @alexlbr, there is a dependency vulnerability that's causing that test to fail. It's not from your code and our team is on it! |
Thanks for the PR! Looks great. Going to get a couple other eyes on this to double check all in the info. But super excited for this one! |
Updates fork
…emy/gatsby into docs/bootstrap-sequence
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.
Thanks @alexlbr this looks great! I added one suggestion based on a recent change in Gatsby.
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.
Thank you so much for this @alexlbr
Left a couple of comments!
docs/docs/gatsby-lifecycle-apis.md
Outdated
- reads and validates `gatsby-config.js` to load in your list of plugins (it doesn't run them yet). | ||
- deletes HTML and CSS files from the previous build (public folder) | ||
- initializes its cache (stored in `/.cache`) and checks if any plugins have been updated since the last run, if so it deletes the cache | ||
- runs the plugins |
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.
runs gatsby-node
- runs the plugins | |
- sets up `gatsby-browser` and `gatsby-ssr` for plugins that have them |
docs/docs/gatsby-lifecycle-apis.md
Outdated
- runs source nodes again and it updates the GraphQL schema since pages can include queries to other data. Notice this is the second time source nodes run due to a "chicken and the egg problem". We need the pages to extract the queries but we need to create the schema first which requires to source nodes | ||
- runs [onPreExtractQueries](/docs/node-apis/#onPreExtractQueries) e.g. implemented by [`gatsby-transformer-sharp`](https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby-transformer-sharp/src/gatsby-node.js) and [`gatsby-source-contentful`](https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby-source-contentful/src/gatsby-node.js), and extracts queries from pages and components (`StaticQuery`) | ||
- compiles queries using a Relay compiler and creates the AST | ||
- applies validation rules in the schema |
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.
- applies validation rules in the schema | |
- runs query validation based on schema |
Syncing upstream repository with the fork
@sidharthachatterjee, @m-allanson, @laurieontech thanks a lot for all the feedback. I've updated the PR with your comments. Sorry that it took a bit to come back to this PR. Thanks a lot again! |
Thank you so much for writing this up @alexlbr 🥇 |
Holy buckets, @alexlbr — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
Description
Addresses Issues
#9350