-
Notifications
You must be signed in to change notification settings - Fork 49.8k
[Fax] Publish experimental_renderToHTML
#30690
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
1ea59a4 to
23d0254
Compare
|
Comparing: 5c9243d...79b705b Critical size changesIncludes critical production bundles, as well as any change greater than 2%:
Significant size changesIncludes any change greater than 0.2%: (No significant changes) |
350b1f3 to
6fd10f3
Compare
| @@ -130,8 +135,20 @@ if (!__EXPERIMENTAL__) { | |||
| } | |||
|
|
|||
| await expect(async () => { | |||
| await ReactHTML.renderToMarkup(React.createElement(Component)); | |||
| }).rejects.toThrow(); | |||
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.
Test would've passed without renaming so I added the exact message we expect.
6fd10f3 to
f7ccf0b
Compare
rickhanlonii
left a comment
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.
🎉
062f774 to
29b5f33
Compare
experimental_renderToHTMLexperimental_renderToHTML
Need to get greenlight from @rickhanlonii first that the bot get the maintainer invitationreact-bot has publish rights.Summary
Publishes
react-markup@experimentalrenderToHTMLis experimental so it should be published with the prefix.Reference docs: reactjs/react.dev#7107
How did you test this change?