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

[gatsby-source-contentful] Import graphql from gatsby (fragments) #6091

Merged
merged 1 commit into from
Jun 22, 2018
Merged

[gatsby-source-contentful] Import graphql from gatsby (fragments) #6091

merged 1 commit into from
Jun 22, 2018

Conversation

ryanditjia
Copy link
Contributor

Fixes the deprecation warnings.

@gatsbybot
Copy link
Collaborator

Deploy preview for using-drupal ready!

Built with commit ed45950

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

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.

Thanks!

This would be the kind of change that's perfectly suited to a codemod. Would you be interested in trying your hand at that?

There's a related issue over at #5038

@gatsbybot
Copy link
Collaborator

Deploy preview for gatsbygram ready!

Built with commit ed45950

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

@m-allanson m-allanson merged commit 5f50317 into gatsbyjs:master Jun 22, 2018
@ryanditjia ryanditjia deleted the patch-1 branch June 25, 2018 08:22
@ryanditjia
Copy link
Contributor Author

I don’t have enough knowledge on Rust and Regex yet, but I’ll take a look when I have time!

@pieh
Copy link
Contributor

pieh commented Jun 25, 2018

@ryanditjia There is already work being done on codemod - #6122 so check that if You are interested in codemods :)

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.

None yet

4 participants