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

Support typescript when parsing GraphQL queries #6335

Merged
merged 2 commits into from
Jul 12, 2018

Conversation

dennari
Copy link
Contributor

@dennari dennari commented Jul 7, 2018

Also extract ast parsing into a utility function

@@ -24,25 +24,7 @@ module.exports = (modulePath, resolver = require.resolve) => {
}
const code = fs.readFileSync(absPath, `utf8`) // get file contents

const babylonOpts = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is now using the larger set of plugins. Hopefully that's not a problem?

@dennari
Copy link
Contributor Author

dennari commented Jul 7, 2018

I'm going to be away for two weeks, so please feel free to modify this PR as needed and merge it in.

@gatsbybot
Copy link
Collaborator

Deploy preview for using-drupal ready!

Built with commit feae623

https://deploy-preview-6335--using-drupal.netlify.com

@gatsbybot
Copy link
Collaborator

Deploy preview for gatsbygram ready!

Built with commit feae623

https://deploy-preview-6335--gatsbygram.netlify.com

@jeffwillette
Copy link
Contributor

its working for me

Copy link
Contributor

@KyleAMathews KyleAMathews left a comment

Choose a reason for hiding this comment

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

Looks good to me!

/cc @jquense @m-allanson @pieh look good?

Copy link
Contributor

@m-allanson m-allanson left a comment

Choose a reason for hiding this comment

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

@dennari Thanks for your patience with seeing this through. This LGTM 👍

@m-allanson m-allanson merged commit cce719d into gatsbyjs:master Jul 12, 2018
@m-allanson
Copy link
Contributor

Published as gatsby@2.0.0-beta.28

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.

5 participants