Plan: Architecture review, and 3 more #57

Manually merged
mdoorn merged 4 commits from feat/057-plan-architecture-review-and-3-more into master 2026-08-14 08:30:00 +02:00
Owner

plan: add 01M1XJZSDGJ90YGY9ZP6HBYPS9-architecture-review

Along the way:

  • plan: add 01M1XJZSQ9RFZCH8SS8YRETP24-eachtexture-into-core-beside-ns-strip
  • plan: add 01M1XJZT13XYG6HMR6VAZ14RX5-core-menu-lua-registers-a-feature-from-inside-co
  • plan: add 01M1XJZTB04N5ZVHFD0S25FYQZ-unitframes-enemybars-lua-is-three-modules-in-one

Touches 4 files: plan/tickets/01M1XJZSDGJ90YGY9ZP6HBYPS9-architecture-review/ticket.md, plan/tickets/01M1XJZSQ9RFZCH8SS8YRETP24-eachtexture-into-core-beside-ns-strip/ticket.md, plan/tickets/01M1XJZT13XYG6HMR6VAZ14RX5-core-menu-lua-registers-a-feature-from-inside-co/ticket.md, plan/tickets/01M1XJZTB04N5ZVHFD0S25FYQZ-unitframes-enemybars-lua-is-three-modules-in-one/ticket.md.

plan: add 01M1XJZSDGJ90YGY9ZP6HBYPS9-architecture-review Along the way: - plan: add 01M1XJZSQ9RFZCH8SS8YRETP24-eachtexture-into-core-beside-ns-strip - plan: add 01M1XJZT13XYG6HMR6VAZ14RX5-core-menu-lua-registers-a-feature-from-inside-co - plan: add 01M1XJZTB04N5ZVHFD0S25FYQZ-unitframes-enemybars-lua-is-three-modules-in-one Touches 4 files: `plan/tickets/01M1XJZSDGJ90YGY9ZP6HBYPS9-architecture-review/ticket.md`, `plan/tickets/01M1XJZSQ9RFZCH8SS8YRETP24-eachtexture-into-core-beside-ns-strip/ticket.md`, `plan/tickets/01M1XJZT13XYG6HMR6VAZ14RX5-core-menu-lua-registers-a-feature-from-inside-co/ticket.md`, `plan/tickets/01M1XJZTB04N5ZVHFD0S25FYQZ-unitframes-enemybars-lua-is-three-modules-in-one/ticket.md`.
mdoorn approved these changes 2026-08-14 08:25:00 +02:00
mdoorn left a comment

An architecture review written as tickets rather than as a document is the only version of this anybody will act on. Two notes.

An architecture review written as tickets rather than as a document is the only version of this anybody will act on. Two notes.
@ -0,0 +11,4 @@
`src/Core/Core.lua:8` promises Core knows nothing about any feature, and
twenty-five parts keep that by registering from `<Folder>/Feature.lua`.
`src/Core/Menu.lua:275` does not, and wants a folder like the rest. Last,
after 20 and 26 have proved the registry needs no help from Core.
Owner

Scheduled last, after the two that prove the registry needs no help from Core. That ordering is the whole value of the ticket and it should be in the title, not in the last line.

Scheduled last, after the two that prove the registry needs no help from Core. That ordering is the whole value of the ticket and it should be in the title, not in the last line.
@ -0,0 +17,4 @@
Reset, Repair and Resolve, inside a nameplate widget.
- `:1482` is 8 functions over `stripped`, `pending` and `passThrough`,
sharing one name with the modes block. The ninth `Blizzard.lua`, filed
under another name inside the file it hides plates for.
Owner

"The ninth Blizzard.lua, filed under another name inside the file it hides plates for." That is the finding. Everything above it is evidence.

"The ninth Blizzard.lua, filed under another name inside the file it hides plates for." That is the finding. Everything above it is evidence.
Author
Owner

Moved into the title.

Moved into the title.
Author
Owner

It was the last thing I found and the first thing anybody needs to read, so it has moved to the top.

It was the last thing I found and the first thing anybody needs to read, so it has moved to the top.
mdoorn manually merged commit 3f6dda138c into master 2026-08-14 08:30:00 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
northgale/warriorkit!57
No description provided.