docs: Add Architecture Decision Records (ADR) directory#6189
Open
ntkathole wants to merge 2 commits intofeast-dev:masterfrom
Open
docs: Add Architecture Decision Records (ADR) directory#6189ntkathole wants to merge 2 commits intofeast-dev:masterfrom
ntkathole wants to merge 2 commits intofeast-dev:masterfrom
Conversation
Introduce an ADR directory (docs/adr/) to keep finalized RFC decisions version-controlled alongside the codebase. This replaces the docs/rfcs/ directory and adds 8 ADRs from approved RFCs that are still applicable to the current codebase: - ADR-0001: Feature Services (RFC-015) - ADR-0002: Component Refactor (RFC-020) - ADR-0003: On-Demand Transformations (RFC-021) - ADR-0004: Entity Join Key Mapping (RFC-023) - ADR-0005: Stream Transformations (RFC-036) - ADR-0006: Kubernetes Operator (RFC-042) - ADR-0007: Unified Feature Transformations (RFC-043) - ADR-0008: Feature View Versioning Also updates contribution guide and governance docs to reference the ADR directory and adds the ADR process to the RFC workflow. Co-Authored-By: Claude Opus 4.6 <[email protected]> Signed-off-by: Ambient Code Bot <[email protected]>
6bc25cd to
165d607
Compare
Member
franciscojavierarceo
left a comment
There was a problem hiding this comment.
i see a few more here: https://drive.google.com/drive/folders/1msUsgmDbVBaysmhBlg9lklYLLTMk4bC3?usp=drive_link
Add three more Architecture Decision Records for approved RFCs that are still applicable to the current codebase: - ADR-0009: Contribution and Extensibility Architecture (RFC-014) - ADR-0010: Vector Database Integration for LLM/RAG Support (RFC-040) - ADR-0011: Data Quality Monitoring (RFC-027) Co-Authored-By: Claude Opus 4.6 <[email protected]> Signed-off-by: Ambient Code Bot <[email protected]>
Member
Author
Added few missing, Some of them are not added as those are Process/governance/documentation structure guide document, not an architectural decision. Rejected/Never finalized or approved approved or very old/not relevant for current code. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
Introduce an ADR directory (docs/adr/) to keep finalized RFC decisions version-controlled alongside the codebase. This replaces the docs/rfcs/ directory and adds 8 ADRs from approved RFCs that are still applicable to the current codebase:
Also updates contribution guide and governance docs to reference the ADR directory and adds the ADR process to the RFC workflow.