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
Roll ICU to c2a4cae149aae7fd30c4cbe3cf1b30df03b386f1 #22465
Conversation
417ea59
to
0669706
Compare
@iskakaushik PTAL. Not sure what I can do about the luci-engine failure. |
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat. Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
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.
LGTM
868d2a1
to
15f0473
Compare
@iskakaushik I will need someone on Flutter team to merge this, assuming that I win the |
Changes to make this work: - Had to change the use of an UBool constant `FALSE` to 0 because the file referring to it does not include the respective header. - Had to change the license checker to ignore files in the directory icu/source/samples. There were random changes to the licensing headers which tripped up license checks. And since we don't include samples in the binaries we build, the most efficient way to fix it is to exclude sample code from license checks. Issue: flutter/flutter#70370
15f0473
to
3846754
Compare
Thank you! |
Changes to make this work: - Had to change the use of an UBool constant `FALSE` to 0 because the file referring to it does not include the respective header. - Had to change the license checker to ignore files in the directory icu/source/samples. There were random changes to the licensing headers which tripped up license checks. And since we don't include samples in the binaries we build, the most efficient way to fix it is to exclude sample code from license checks. Issue: flutter/flutter#70370
Issue: flutter/flutter#70370
Description
Related Issues
Tests
N/A, this is a library dependency update.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.Reviewer Checklist
Breaking Change
Did any tests fail when you ran them? Please read handling breaking changes.