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

chore: e2e/integration with React 17 & 18 #35320

Merged
merged 13 commits into from Apr 7, 2022
Merged

Conversation

LekoArts
Copy link
Contributor

@LekoArts LekoArts commented Apr 1, 2022

Description

Runs development-runtime and production-runtime with React 18 in addition to React 17.

This PR also removes the outdated nightly jobs (no one looked at them + the slack setup was removed).

It also updates all e2e/integration tests to run with React 17.

[ch48484]

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Apr 1, 2022
@LekoArts LekoArts added type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change topic: automation Related to Circle CI, Peril, Renovate, scripts/*, Github Workflows, Github Actions, or Slackbot and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Apr 1, 2022
@LekoArts LekoArts changed the title chore: Run development/prod e2e with React 18 chore: e2e/integration with React 17 & 18 Apr 1, 2022
@LekoArts
Copy link
Contributor Author

LekoArts commented Apr 6, 2022

Ok, this has been a deep rabbit hole.

The production runtime tests failed with the lifecycle tests because render occurred twice on the initial serve of a page. This is because React sees a Hydration error and re-tries the page.

The hydration error doesn't occur in minimal reproduction project, only when you use StaticImage or GatsbyImage with the fixed layout option.

For now these tests will be changed with a TODO and #35226 then needs to actually fix the image error and revert the test changes.

@LekoArts LekoArts marked this pull request as ready for review April 6, 2022 13:15
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

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

Let's go! Looks good

@LekoArts LekoArts merged commit bb3287e into master Apr 7, 2022
@LekoArts LekoArts deleted the run-e2e-with-react-18 branch April 7, 2022 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: automation Related to Circle CI, Peril, Renovate, scripts/*, Github Workflows, Github Actions, or Slackbot type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants