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

fix(gatsby): wait for app-data.json file writing #22099

Merged
merged 3 commits into from
May 13, 2020

Conversation

MichaelDeBoey
Copy link
Contributor

Follow-up of #22076

@MichaelDeBoey MichaelDeBoey force-pushed the cleanup/gatsby/utils/app-data branch from 272ef1a to c095a12 Compare March 9, 2020 16:06
@blainekasten
Copy link
Contributor

I need to manually pull this down and test. This seems like the intended behavior given that we await it, but I want to verify nothing breaks. ❤️

@MichaelDeBoey
Copy link
Contributor Author

@pieh pieh self-assigned this May 5, 2020
@pieh
Copy link
Contributor

pieh commented May 11, 2020

I'll do some changes to avoid changes that are subjective (they don't change anything in practice, because it's still handled by babel and executed in Node.js context, so this really just change syntax and I would argue that fs.outputJson is easier for grepping etc than outputJson) and only add fixed return type (void -> Promise)

@pieh pieh changed the title Cleanup app-data fix(gatsby): wait for app-data.json file writing May 11, 2020
@pieh pieh merged commit 037de56 into gatsbyjs:master May 13, 2020
@MichaelDeBoey MichaelDeBoey deleted the cleanup/gatsby/utils/app-data branch May 13, 2020 09:23
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.

3 participants