Skip to content

Conversation

stephanos
Copy link
Contributor

What changed?

Describe what has changed in this PR.

Why?

Tell your future self why have you made these changes.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Potential risks

Any change is risky. Identify all risks you are aware of. If none, remove this section.

@stephanos stephanos force-pushed the customizable-serialization-2 branch 4 times, most recently from 983f58c to 9edd884 Compare October 3, 2025 15:59
@yycptt
Copy link
Member

yycptt commented Oct 3, 2025

Is this PR same as #8222?

@stephanos
Copy link
Contributor Author

stephanos commented Oct 3, 2025

@yycptt I left the actual customization in this one and will rebase it once #8222 is merged - that one only contains the refactorings now. The bulk of the changes are in #8222 so it's easier to complete, merge and review them separately IMO.

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.

2 participants