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

[Flight] Enable Binary and ReadableStreams in Stable #29035

Merged
merged 1 commit into from
May 10, 2024

Conversation

sebmarkbage
Copy link
Collaborator

These are ready to ship in stable.

@sebmarkbage sebmarkbage requested a review from rickhanlonii May 9, 2024 16:18
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels May 9, 2024
@@ -72,8 +72,8 @@ export const renameElementSymbol = false;
export const enableCache = true;
export const enableLegacyCache = true;

export const enableBinaryFlight = false;
export const enableFlightReadableStream = false;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since HaaS doesn't support binary data in the stream itself, it'd need something in the stream to encode it as a string but it's not really different from how an SSR implementation has to deal with it.

@react-sizebot
Copy link

The size diff is too large to display in a single comment. The CircleCI job contains an artifact called 'sizebot-message.md' with the full message.

Generated by 🚫 dangerJS against 16efc82

Copy link
Member

@rickhanlonii rickhanlonii left a comment

Choose a reason for hiding this comment

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

SGTM

@sebmarkbage sebmarkbage merged commit 6ef0dd4 into facebook:main May 10, 2024
38 checks passed
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.

4 participants