Skip to content

Conversation

@JonathanOppenheimer
Copy link
Member

Why this should be merged

A lot of this code has been rewritten, and there is no longer a risk of having a breaking change with them anymore -- it doesn't make sense to maintain this old code any longer.

Closes #1902

How this was tested

CI

Need to be documented?

No

Need to update RELEASES.md?

No

Copy link
Collaborator

@ceyonur ceyonur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice cleanup!

@ceyonur ceyonur added this pull request to the merge queue Dec 4, 2025
Merged via the queue into master with commit 6602ce7 Dec 4, 2025
15 checks passed
@ceyonur ceyonur deleted the JonathanOppenheimer/remove-old-pack-unpack branch December 4, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove old unpack/pack code

3 participants