docs(rfc): mark RFC 0029 accepted and drop the private-repo citation - #445
Open
aaltshuler wants to merge 1 commit into
Open
docs(rfc): mark RFC 0029 accepted and drop the private-repo citation#445aaltshuler wants to merge 1 commit into
aaltshuler wants to merge 1 commit into
Conversation
Two lifecycle items for the merge of #440: - docs/rfcs/README.md states maintainers own the Status field and it becomes Accepted on merge for the public-contribution track; the merged file still said Proposed. - The diagram prompt cited a workflow repository that is not publicly reachable, which AGENTS.md maintenance rule 5 (keep docs audience-neutral) asks to translate into a durable public rationale. The prompt below the note is already self-contained, so the attribution is replaced with a neutral description of the method. Raised in review on #440.
| | | | | ||
| |---|---| | ||
| | **Status** | Proposed | | ||
| | **Status** | Accepted (2026-08-05) | |
There was a problem hiding this comment.
Avoid an undocumented status variant
The RFC process defines the merged status as bare Accepted; adding a parenthesized date creates an undocumented variant and leaves exact status interpretation and the convention for future RFCs ambiguous.
Suggested change
| | **Status** | Accepted (2026-08-05) | | |
| | **Status** | Accepted | |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
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
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.
Follow-up to #440 (merged as
12a85966).Two lifecycle items that belong with the merge rather than with later design adjustments:
docs/rfcs/README.mdspecifies that maintainers own the Status field and that it readsAcceptedon merge for the public-contribution track. The merged file still saidProposed.msftse/azure-architecture-proposal, which an outside reader cannot open — the case AGENTS.md maintenance rule 5 asks to translate into a durable public rationale. The prompt text below the note is already fully self-contained, so the attribution becomes a neutral description of the method. This was raised in review on RFC 0029: native Azure Blob storage #440; the thread was resolved to unblock the merge on the commitment to fix it here.Docs-only.
scripts/check-agents-md.shpasses (74 links, 69 docs).Greptile Summary
This documentation-only PR marks RFC 0029 as accepted and replaces an inaccessible private-repository citation with an audience-neutral description.
Confidence Score: 4/5
The PR appears safe to merge, with a non-blocking RFC metadata consistency issue to resolve.
The prompt remains self-contained and audience-neutral, while the RFC status introduces an undocumented parenthesized-date variant rather than using the defined bare
Acceptedvalue.Files Needing Attention: docs/rfcs/0029-azure-blob-storage.md
Important Files Changed
Reviews (1): Last reviewed commit: "docs(rfc): mark RFC 0029 accepted and dr..." | Re-trigger Greptile