Skip to content

Remove .internal from more test suites#18597

Merged
acdlite merged 1 commit into
facebook:masterfrom
acdlite:remove-internal
Apr 14, 2020
Merged

Remove .internal from more test suites#18597
acdlite merged 1 commit into
facebook:masterfrom
acdlite:remove-internal

Conversation

@acdlite
Copy link
Copy Markdown
Collaborator

@acdlite acdlite commented Apr 14, 2020

This enables the tests to run against the actual build artifacts

@acdlite acdlite marked this pull request as draft April 14, 2020 03:22
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Apr 14, 2020
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci Bot commented Apr 14, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9f000d3:

Sandbox Source
purple-https-c79dl Configuration

@sizebot
Copy link
Copy Markdown

sizebot commented Apr 14, 2020

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 9f000d3

@sizebot
Copy link
Copy Markdown

sizebot commented Apr 14, 2020

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 9f000d3

@acdlite acdlite force-pushed the remove-internal branch 2 times, most recently from 2ada58c to 26033ed Compare April 14, 2020 06:51
Comment on lines +60 to +61
export const scheduleTimeout = $$$hostConfig.scheduleTimeout;
export const cancelTimeout = $$$hostConfig.cancelTimeout;
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Welp turns out react-reconciler didn't work with Suspense!

@acdlite acdlite marked this pull request as ready for review April 14, 2020 06:59
@acdlite acdlite merged commit 843b50c into facebook:master Apr 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants