Markus Scherer
Markus Scherer
> > Is `unisift_IsIgnorable` intentionally different from Default_Ignorable_Code_Point? > > Yes. The "Only in A" list is mostly [:Cn:] except for 7 other characters. Try `[:DI:]-[:Cn:]` for A.
> So, now that I can run the sifter, I still need to update unidata.txt for it to actually do something useful. > > I gather this is the relevant...
changes lgtm > If I am reading the draft SAH recommendations right we should split out the GC change for 16.0. No, the consensus is for changing the gc in...
Is it comparing String values? It should compare VersionInfo values. (I believe that the ICU UnicodeSet implementation handles Age properly.)
I had to rebase in order to make a newer CI check work. Now we find out if it still works with the latest checks, even pre-merge...
@FrankYFTang setting you as the main reviewer because Mihai is ooo for a while soon. Note that the API change proposal is still under discussion.
Idea: Consider changing LBLetters(Hani) to "No" but adding WBLetters and making that "Yes" for Hani.
I see that there was a question about whether there is a boolean-returning API for “LB Letters”. The answer is yes, in both ICU4C & ICU4J. https://unicode-org.github.io/icu-docs/apidoc/released/icu4j/com/ibm/icu/lang/UScript.html#breaksBetweenLetters-int-
> This file still needs to be updated: icu4j/releases_tools/shared.sh > > Contains an `rc` (`export github_rel_version='76rc'`) Rahul _is_ updating that in this PR, to `github_rel_version='76_1'`. Looks like this is complete...
PS: Please don't squash until after approval, except when you need to rebase.