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-cli,gatsby-recipes): only run recipes when admin is enabled #26671

Closed
wants to merge 3 commits into from

Conversation

wardpeet
Copy link
Contributor

Description

When closing develop sometimes the recipes GraphQL wouldn't exit and stay in limbo. This uses execa to make sure we kill the process. I also added an env var so it only loads with admin.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 27, 2020
@vladar vladar removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 28, 2020
@wardpeet wardpeet added status: needs core review Currently awaiting review from Core team member topic: recipes labels Aug 31, 2020
laurieontech
laurieontech previously approved these changes Aug 31, 2020
Copy link
Contributor

@laurieontech laurieontech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is still the chance I'm missing something here but this should be fine.

@LekoArts LekoArts added topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) and removed topic: recipes labels May 28, 2021
@LekoArts
Copy link
Contributor

LekoArts commented Dec 8, 2021

Closing due to #34094

@LekoArts LekoArts closed this Dec 8, 2021
@wardpeet wardpeet deleted the fix/recipes-kill branch December 9, 2021 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs core review Currently awaiting review from Core team member topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants