Add Applier/Operator components and combining rule - #209
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 12 included reviews per hour; 9 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughThe ticket hygiene standard separates domain and cross-cutting components. It adds Applier and Operator as domain components. It limits Infra to Helm and deployment responsibilities. It adds rules for combining components across domains and work types. Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a localized documentation change with no actionable merge-blocking risk remaining after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Adds Applier (hyperfleet-applier) and Operator (hyperfleet-operator) to the valid components list — both repos previously had no matching component and their work was mislabeled under Infra/Adapter/Architecture. Splits the Valid Components table into domain (which system) and cross-cutting (which kind of work) groups, and documents when to combine one of each (e.g. Applier + Architecture for a domain-specific design spike, Operator + Documentation for a CR reference guide).
b9cd85e to
ca17f4d
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kuudori The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Last Updated is still 2026-05-20. Worth bumping since this adds new components and a new subsection. |
|
PR titles need a JIRA prefix per the commit standard. Add a ticket reference if one exists. |
Summary
Applier(hyperfleet-applier) andOperator(hyperfleet-operator) to the valid components table — both repos had no matching component, so their work was mislabeled under Infra/Adapter/ArchitectureApplier+Architecturefor a domain-specific design spike,Operator+Documentationfor a CR reference guide)Test plan