Skip to content

Conversation

@thantos
Copy link
Collaborator

@thantos thantos commented Sep 16, 2022

#402 on top of #510

sam and others added 30 commits August 2, 2022 00:37
Signed-off-by: github-actions <github-actions@github.com>
@netlify
Copy link

netlify bot commented Sep 16, 2022

Deploy Preview for effortless-malabi-1c3e77 ready!

Name Link
🔨 Latest commit 9778b48
🔍 Latest deploy log https://app.netlify.com/sites/effortless-malabi-1c3e77/deploys/63249ede29deba000887d187
😎 Deploy Preview https://deploy-preview-511--effortless-malabi-1c3e77.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@sam-goodwin
Copy link
Owner

Should we point this at my original PR and merge it?

@sam-goodwin
Copy link
Owner

I would like to see the changes between this and what's in mine. Merging against main makes that hard to see.

@sam-goodwin
Copy link
Owner

Oh never mind - I misunderstood the purpose of this change. Looks like closure serializer is stable. I have one more commit to make where I put the default serializer back to v1. Then I'll merge.

@thantos
Copy link
Collaborator Author

thantos commented Sep 16, 2022

#402 has been updated with this change, closing.

@thantos thantos closed this Sep 16, 2022
@thantos thantos deleted the sussman/serialize-perf branch September 16, 2022 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants