Overview
We have 7 open PRs from @Nantero1 adding new semantic anchors. Before merging, each needs to be reviewed for quality, structure, and completeness.
Open PRs
| PR |
Anchor |
Category (to verify) |
| #137 |
CQRS |
Software Architecture? |
| #138 |
BDD (Given-When-Then) |
Testing & Quality? |
| #139 |
YAGNI |
Design Principles? |
| #140 |
GoF Design Patterns |
Design Principles? |
| #141 |
Chain of Responsibility |
Design Principles? |
| #142 |
Test Double (Meszaros) |
Testing & Quality? |
| #143 |
Event-Driven Architecture |
Software Architecture? |
Review checklist per PR
Open questions
-
Merge first, then fix? Or fix before merge?
- Pro merge-first: Less work for the contributor, we can batch-fix translations
- Pro fix-first: Cleaner history, no broken state on main
- Suggestion: Review content quality and anchor suitability first (reject/request changes if needed), then merge acceptable ones and batch-add translations ourselves.
-
Anchor suitability concerns:
-
Consistency with existing anchors: Do the new ones follow the same depth and style as our existing 48 anchors?
Suggested workflow
- Review each PR for anchor quality (comment on PR if changes needed)
- Merge approved PRs one by one
- Batch-create German translations for all merged anchors
- Verify website builds correctly after each merge
Overview
We have 7 open PRs from @Nantero1 adding new semantic anchors. Before merging, each needs to be reviewed for quality, structure, and completeness.
Open PRs
Review checklist per PR
.de.adocfiles.Open questions
Merge first, then fix? Or fix before merge?
Anchor suitability concerns:
Consistency with existing anchors: Do the new ones follow the same depth and style as our existing 48 anchors?
Suggested workflow