Skip to content
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

Hani CJK strokes #836

Merged
merged 4 commits into from
May 24, 2024
Merged

Hani CJK strokes #836

merged 4 commits into from
May 24, 2024

Conversation

@eggrobin eggrobin requested review from markusicu and roozbehp May 24, 2024 13:01
@eggrobin eggrobin merged commit af5b2d9 into unicode-org:main May 24, 2024
15 checks passed
Copy link
Contributor

@roozbehp roozbehp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

eggrobin added a commit to eggrobin/unicodetools that referenced this pull request May 31, 2024
eggrobin added a commit to eggrobin/unicodetools that referenced this pull request May 31, 2024
… new comparison test fails"

This reverts commit 04cc2b0.
eggrobin added a commit that referenced this pull request May 31, 2024
* This should not be a test but it seems useful.

* null

* blarg

* blaaargh

* Put the . in the extension

Co-authored-by: Markus Scherer <[email protected]>

* suffix for testInvariants

* docs

* Propertywise … AreAlike in the invariant language

* Revert "Hani CJK strokes (#836)" to check that the new comparison test fails

This reverts commit af5b2d9.

* Another reference

* Remove early eggsperiment

* Remove trace

* Revert "Revert "Hani CJK strokes (#836)" to check that the new comparison test fails"

This reverts commit 04cc2b0.

* Placeholder heading

* Don’t compare first with itself.

Co-authored-by: Markus Scherer <[email protected]>

* no inner loop

---------

Co-authored-by: Markus Scherer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants