-
Notifications
You must be signed in to change notification settings - Fork 35
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
B-21667 & B-21487 MAIN #14288
B-21667 & B-21487 MAIN #14288
Conversation
Bundle StatsHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded
Removed
Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
Heads up I resolved a lot of these conflicts in the upstream @traskowskycaci , don't worry about resolving them until the upstream is merged. Once merged the fix will be in the git history |
Alternatively you can merge the upstream B-21374 into your PR now and it'll resolve them like that, I did that just now with #14342 |
I'll merge in 21374 - just brought it in yesterday to resolve some test fails so shouldn't be too bad! Thanks for the heads up @cameroncaci |
#14100 upstream to this pr is next, advise merging it in to help resolve the current conflicts |
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.
Both tickets POA'd. All three issue tickets present. Commit shas are GOOD. WHEEEW 🚀
Important
Please ensure the upstreams are merged into main first:
B-21374
B-21375
B-21376
This main branch contains commits from all the below integration PRs. I combined B-21667 and B-21487, because the rename of a variable in B-21487 was the cause of some of the issues needing fixed.
B-21667 INT PR
B-21487 INT PR
B-21667 I-13550 INT PR
B-21667 I-13567 INT PR
B-21667 I-13574 INT PR