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 V3 #146

Merged
merged 10 commits into from
Mar 21, 2021
Merged

Gatsby V3 #146

merged 10 commits into from
Mar 21, 2021

Conversation

tnorlund
Copy link
Contributor

I'd really like to get this working with Gatsby V3. I'm having issues with the Jest suite. I'm not sure how to generate another snapshot in __snapshots__. Could you help me out?

@tnorlund tnorlund mentioned this pull request Mar 20, 2021
Copy link
Owner

@andrewbranch andrewbranch left a comment

Choose a reason for hiding this comment

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

Thanks for the help! Running yarn test -u should update all the baselines. There are some instructions in CONTRIBUTING.md.

@tnorlund
Copy link
Contributor Author

All of the GraphQL resolvers are failing. Does the yarn test -u change the expectation of the tests?

@andrewbranch
Copy link
Owner

My bad, react is now a peerDependency of gatsby, so it does need to be installed. The issue is that gatsby's local server (used to test GraphQL queries) isn't spinning up at all. I'm taking a look and will push an update.

@andrewbranch
Copy link
Owner

@tnorlund, can you check the box on the PR that allows maintainers to make edits? That will let me push a commit.

@tnorlund
Copy link
Contributor Author

@tnorlund, can you check the box on the PR that allows maintainers to make edits? That will let me push a commit.

It's already checked. Do you want me to uncheck it?

@andrewbranch
Copy link
Owner

Oh, I think it's because it's on the master branch of your fork. I think the easiest thing to do would be to give me push access to your fork. Otherwise I'll have to PR into your PR.

@tnorlund
Copy link
Contributor Author

I added you as a collaborator. Does that work?

@andrewbranch
Copy link
Owner

That worked, thanks!

Copy link
Owner

@andrewbranch andrewbranch left a comment

Choose a reason for hiding this comment

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

Ok, tests are working, which is a good sign. I think before merging, I would like to get the example site updated to v3 as well just so we can see that it works fully integrated in a real gatsby environment. Would you mind looking into that? I really don’t care whether it’s upgraded in place or recreated with gatsby new and the content and config dropped back in. Whatever you think is easier, if you’re up for it.

@tnorlund
Copy link
Contributor Author

Are you referring to the site in examples/example-site/?

@andrewbranch
Copy link
Owner

Yep, that’s the one.

@tnorlund
Copy link
Contributor Author

Alright. I'm going to make the changes required for Gatsby V3.

@tnorlund
Copy link
Contributor Author

Tests pass locally, fail in GH Action. Any idea why?

@tnorlund
Copy link
Contributor Author

If I can help with anything else, please let me know!

@tnorlund
Copy link
Contributor Author

I guess that's it. I'd like to add this to my blog once it's been merged. It's a neat plugin.

@andrewbranch
Copy link
Owner

Thanks so much!

@andrewbranch andrewbranch merged commit bb3f98c into andrewbranch:master Mar 21, 2021
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.

2 participants