Skip to content

delete unused Main class dispatch options & related code#1435

Merged
markusicu merged 12 commits into
unicode-org:mainfrom
markusicu:delete-unused-main-dispatch
Jun 4, 2026
Merged

delete unused Main class dispatch options & related code#1435
markusicu merged 12 commits into
unicode-org:mainfrom
markusicu:delete-unused-main-dispatch

Conversation

@markusicu

@markusicu markusicu commented Jun 3, 2026

Copy link
Copy Markdown
Member

Fixes #1401

See

(repeat link to get a readable title)

Best reviewed one commit at a time.
I intend to rebase+merge these separate commits.

markusicu added 2 commits June 3, 2026 16:50
Only called from UCA.Main unused dispatch options.
@markusicu markusicu force-pushed the delete-unused-main-dispatch branch from f2cf8eb to 7530cc8 Compare June 4, 2026 02:53
@markusicu markusicu marked this pull request as ready for review June 4, 2026 17:24

@macchiati macchiati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no objections

@markusicu markusicu merged commit 83cde70 into unicode-org:main Jun 4, 2026
27 checks passed
@markusicu markusicu deleted the delete-unused-main-dispatch branch June 4, 2026 21:41
@eggrobin

eggrobin commented Jun 4, 2026

Copy link
Copy Markdown
Member

That is a Δ to celebrate:
image

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.

review/delete code called from UCA/Main & UCD/Main dispatch

4 participants