Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(gatsby-source-graphql): remove graphql dependency and use gatsby/graphql #27150

Merged
merged 2 commits into from
Sep 28, 2020

Conversation

vladar
Copy link
Contributor

@vladar vladar commented Sep 28, 2020

Description

This PR removes graphql dependency from gatsby-source-graphql. Instead, we import from gatsby/graphql directly:

- const { parse } = require(`graphql`)
  const { parse } = require(`gatsby/graphql`)

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Sep 28, 2020
@vladar vladar removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Sep 28, 2020
@wardpeet wardpeet added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Sep 28, 2020
@gatsby-cloud
Copy link

gatsby-cloud bot commented Sep 28, 2020

Gatsby Cloud Build Report

gatsby

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 20m

@gatsbybot gatsbybot merged commit 2e2e5c5 into master Sep 28, 2020
@delete-merged-branch delete-merged-branch bot deleted the vladar/graphql-deps-fix branch September 28, 2020 11:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants