-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Combine rss shrinker and base check lazy imports #34483
Conversation
Go Package Import DifferencesBaseline: 407d013
|
Test changes on VMUse this command from test-infra-definitions to manually test this PR changes on a VM: inv aws.create-vm --pipeline-id=57080008 --os-family=ubuntu Note: This applies to commit 6ddccf2 |
Uncompressed package size comparisonComparison with ancestor Diff per package
Decision |
:seesomethingsaysomething: Your PR is a tiny bit behind, is that expected? |
@@ -0,0 +1,109 @@ | |||
// Unless explicitly stated otherwise all files in this repository are licensed |
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.
Could we update the CODEOWNERS list so the right team own this.
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.
@hush-hush I realized too late that this PR is based on some old changes. if the codeowners is still out of date in main
, you may be in a better position to update it than me. I don't know who the new owner is supposed to be ;)
Closing to avoid confusion, will open a new one based on more recent main state. |
What does this PR do?
I set this to draft too late, sorry about reviewer pings!
Motivation
Describe how you validated your changes
Possible Drawbacks / Trade-offs
Additional Notes