-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
ref: Streamline SDK metadata handling #10251
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once upon a time #4196
This is just way better, so let's stick with it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3rd time's a charm I guess 😅 (#7404)
I agree with Abhi, if the size hit isn't worse than in the last attempts, it makes sense to deduplicate this.
Just ~6000 issues/PRs later! xD |
size-limit report 📦
|
|
6ba6bf9
to
8f5beb9
Compare
8f5beb9
to
651c39a
Compare
OK, I managed to cut it down as much as I could in size! |
There have been tries to do this before, but let's see how things stand today...