-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Universe: use Relay 13 and the new Rust Compiler #2119
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
April 13, 2021 15:39
efa7cbd
to
1ad374d
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
April 13, 2021 15:39
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
April 30, 2021 01:52
1ad374d
to
9067987
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
April 30, 2021 01:52
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
April 30, 2021 02:10
9067987
to
4dc7202
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
April 30, 2021 02:10
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
May 1, 2021 16:36
4dc7202
to
62bab7a
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
May 1, 2021 16:36
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
June 25, 2021 18:34
62bab7a
to
1d0da05
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
June 25, 2021 18:34
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
September 3, 2021 01:01
1d0da05
to
082f2e8
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
September 3, 2021 01:01
Inactive
vercel
bot
temporarily deployed
to
Preview – universe-example-relay
September 3, 2021 01:01
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
September 3, 2021 01:16
082f2e8
to
0fe26fd
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
September 3, 2021 01:17
Inactive
vercel
bot
temporarily deployed
to
Preview – universe-example-relay
September 3, 2021 01:17
Inactive
mrtnzlml
commented
Sep 3, 2021
src/abacus-backoffice/src/products/__generated__/ProductsCardsInCategoryQuery.graphql.js
Show resolved
Hide resolved
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
November 13, 2021 04:45
0fe26fd
to
5e61f60
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
November 13, 2021 04:45
Inactive
vercel
bot
temporarily deployed
to
Preview – universe-example-relay
November 13, 2021 04:45
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
November 13, 2021 04:48
5e61f60
to
ebca323
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
January 6, 2022 20:23
Inactive
mrtnzlml
changed the title
Universe: use a new Relay (Rust) Compiler
Universe: use Relay 13 and the new Rust Compiler
Jan 6, 2022
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
January 6, 2022 20:27
9cfdd12
to
e4fec04
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
January 6, 2022 20:27
Inactive
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
January 6, 2022 20:33
e4fec04
to
461031b
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
January 6, 2022 20:33
Inactive
mrtnzlml
commented
Jan 6, 2022
"relay-runtime": "^12.0.0" | ||
"react-relay": "^13.0.0", | ||
"relay-compiler": "^13.0.0", | ||
"relay-config": "0.0.0-main-d92b9e55", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will remove Relay Config together with the fetch-schema.js
bin removal. This config dependency is useless in Relay 13+
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
January 6, 2022 21:03
461031b
to
4a805c6
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
January 6, 2022 21:03
Inactive
This was referenced Jan 7, 2022
This commit upgrades Relay to version 13 and switches from the old Relay Compiler to the new Rust one (since both things go together). Basically, the main change is that now we have only one Relay config for the whole monorepo and the compiler is being executed for the whole monorepo as well (while being much faster). Additionally, Relay support is directly integrated into Flow so in many cases I simply removed previous Flow types (see `useLazyLoadQuery` and `useFragment`). There is still ongoing effort to improve the Flow types in Relay so not everything is finalized. For this reason I decided to use "Compat" types mode. Similarly, some hooks (`useMutation` and `usePreloadedQuery` for example) still require explicit types information so I didn't change these yet. Regardless of that, we are pretty close to use "Final" types. We just need to wait for the Relay team to finish everything. Many issues were already resolved but there are still some that need to be fixed (not blocking this PR): - facebook/relay#3700 - relayjs/eslint-plugin-relay#131 - prettier/prettier#6102 Important links with additional information: - https://relay.dev/blog/2021/12/08/introducing-the-new-relay-compiler/ - https://github.com/facebook/relay/releases/tag/v13.0.0 - https://github.com/facebook/relay/releases/tag/v13.0.0-rc.0 - https://github.com/facebook/relay/releases/tag/v13.0.0-rc.1 - https://github.com/facebook/relay/releases/tag/v13.0.0-rc.2
mrtnzlml
force-pushed
the
universe_new_relay_compiler
branch
from
January 7, 2022 21:26
4a805c6
to
dc0ca9e
Compare
vercel
bot
temporarily deployed
to
Preview – universe-sx-tailwind-website
January 7, 2022 21:27
Inactive
tbergquist-godaddy
approved these changes
Jan 7, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit upgrades Relay to version 13 and switches from the old
Relay Compiler to the new Rust one (since both things go together).
Basically, the main change is that now we have only one Relay config
for the whole monorepo and the compiler is being executed for the whole
monorepo as well (while being much faster). Additionally, Relay support
is directly integrated into Flow so in many cases I simply removed
previous Flow types (see
useLazyLoadQuery
anduseFragment
).There is still ongoing effort to improve the Flow types in Relay so not
everything is finalized. For this reason I decided to use "Compat" types
mode. Similarly, some hooks (
useMutation
andusePreloadedQuery
forexample) still require explicit types information so I didn't change
these yet. Regardless of that, we are pretty close to use "Final" types.
We just need to wait for the Relay team to finish everything.
Many issues were already resolved but there are still some that need to
be fixed (not blocking this PR):
Important links with additional information: