Skip to content
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

Split out the zstandard change with history. #6166

Open
wants to merge 70 commits into
base: master
Choose a base branch
from

Conversation

rebrowning
Copy link
Contributor

@rebrowning rebrowning commented Mar 15, 2024

Looks like as part of the rebase to update the branch it added my name as well, but the complete history should be there now.

splitting out the zstandard work done by @guzzijones in #5995

If we would rather just have the original PR go through, lmk and I can close this. But I saw a comment on his PR that it'd be nice to split the two out.

All credit for work on this goes to @guzzijones , I'm splitting it out with the hope of moving it through so we can mitigate the document size errors.

@pull-request-size pull-request-size bot added the size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several. label Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL PR that changes 1000+ lines. You should absolutely split your PR into several.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants