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

feat(relay): use relay-compiler v5, fix #11536 #15963

Closed
wants to merge 1 commit into from

Conversation

sibelius
Copy link

Description

This uses relay-compiler v5 instead of gatsby fork

Related Issues

#11536

I've tried to run all tests locally, but there are a lot of them breaking (yarn jest)

@sibelius sibelius requested a review from a team as a code owner July 21, 2019 23:25
@freiksenet
Copy link
Contributor

Sadly Relay have regressed on the bugs that we fixed in their code, Relay should be fixed before we can use it.

@sibelius
Copy link
Author

is there an issue there about it?

@sidharthachatterjee
Copy link
Contributor

@sibelius Not sure about an issue there at the moment but I think @freiksenet is referring to the fixes he made in facebook/relay#2638.

@freiksenet Want to add a test asserting the behaviour that we depend on to better track this?

Closing this in the mean time but thank you for opening this @sibelius and apologies for the confusion here!

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.

3 participants