Move and share shell TextEncoder/TextDecoder polyfill#173
Conversation
✅ Deploy Preview for webkit-jetstream-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Will also be useful for WebKit#170
7f080a0 to
78eea34
Compare
|
Ok, a bit nit-picky: instead of |
|
I'm not sure why jsc/Safari are failing on CI, it does not locally on my x64 Linux machine. The log at https://github.com/WebKit/JetStream/actions/runs/17401099724/job/49401838880?pr=173#step:8:39 seems to already have an error when jsvu installs and smoke-tests jsc @kmiller68 can you reproduce or would know what this could be? Edit: Since #172 CI fails with jsc with the same error (https://github.com/WebKit/JetStream/actions/runs/17403780714/job/49402823507?pr=172), and that doesn't even change any code that is run during testing, this looks like an error on JSC side. |
|
JSC seems to be fine again, and Safari CI is failing since multiple weeks, so merging. |
Just to reduce some redundancy, and might also be useful for #170 (where another TextDecoder polyfill was about to be added.)