-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs: fix extra HTML end tags in docs build #9477
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
|
Build successful! 🎉 |
snowystinger
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.
why is this actually happening? is it a bug in rsc-html-stream? or does that just make the existing issue worse?
|
Build successful! 🎉 |
|
Build successful! 🎉 |
|
Build successful! 🎉 |
devongovett
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.
Let's merge this with the verification script for a few days and see if it catches anything else. If not, we'll update rsc-html-stream with this patch.
|
Looks like that didn't fix it: https://app.circleci.com/pipelines/github/adobe/react-spectrum/31023/workflows/21159712-3276-4bc4-8cae-6bfd1c7bcb2a/jobs/568803 We can keep the script around until we do. |
This is difficult to consistently reproduce, but occasionally in the S2 docs build, an extra
</script></body></html>is getting left in the HTML output.Example in production at: https://react-aria.adobe.com/examples/photos
(View page source and search
</script></body></html>)This seems like it might be due to the trailer being split across chunk boundaries in rsc-html-stream. This PR applies the fix from devongovett/rsc-html-stream#12 as a patch and adds a script that will fail the docs build if it detects the issue again.
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: