Skip to content

Clarify Final Comment Period status in the core/README - #1979

Open
anupsdf wants to merge 1 commit into
stellar:masterfrom
anupsdf:readme-update
Open

Clarify Final Comment Period status in the core/README#1979
anupsdf wants to merge 1 commit into
stellar:masterfrom
anupsdf:readme-update

Conversation

@anupsdf

@anupsdf anupsdf commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

what

Clarify Final Comment Period: Acceptance or Rejection in the core/README.

why

To remove ambiguity around the status of CAPs.

Copilot AI review requested due to automatic review settings July 14, 2026 18:24
@anupsdf
anupsdf enabled auto-merge (squash) July 14, 2026 18:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Clarifies the CAP lifecycle wording in core/README.md by making the Final Comment Period (FCP) explicitly indicate its intended disposition (Acceptance vs Rejection), reducing ambiguity in CAP statuses across the spec docs.

Changes:

  • Replaces the ambiguous “FCP: [Acceptance/Rejection]” terminology with “Final Comment Period: [Acceptance/Rejection]” and clarifies the transition outcomes.
  • Updates the CAP-0085 entry in core/README.md to use the new explicit FCP disposition.
  • Synchronizes CAP-0085’s preamble Status field with the README index entry.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
core/README.md Clarifies workflow/status terminology and updates CAP-0085’s indexed status to include intended FCP disposition.
core/cap-0085.md Updates the CAP preamble Status to match the README’s explicit “Final Comment Period: Acceptance” status.

Comment thread core/README.md Outdated
Comment thread core/README.md Outdated
@MonsieurNicolas

Copy link
Copy Markdown
Contributor

I would just replace it with "Final Comment Period" as this is how we've been using in the past and how it's referenced in the document anyways

@anupsdf

anupsdf commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

I would just replace it with "Final Comment Period" as this is how we've been using in the past and how it's referenced in the document anyways

Removed the disposition from the FCP status. Those will now have to be inferred from the discussion thread.

@anupsdf anupsdf changed the title Clarify Final Comment Period: Acceptance or Rejection in the core/README Clarify Final Comment Period status in the core/README Jul 14, 2026
Comment thread core/README.md Outdated
@anupsdf
anupsdf marked this pull request as draft July 27, 2026 19:32
auto-merge was automatically disabled July 27, 2026 19:32

Pull request was converted to draft

@anupsdf
anupsdf marked this pull request as ready for review July 27, 2026 20:40
@anupsdf
anupsdf enabled auto-merge (squash) July 27, 2026 20:40
Comment thread core/README.md
intended disposition. After one week has passed, during which any new concerns should be
addressed, the CAP will head towards its intended disposition [**Acceptance/Rejection**] or go
back into a Draft state.
intended disposition of Acceptance or Rejection for the Final Comment Period, or go back into a

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the reference to "Acceptance or Rejection" is misleading the next state is called "Final Comment Period" or "Draft"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, its not very clear. The way I understood this workflow is that before rejecting a CAP, we want it to move to Final Comment Period for a week. Since, we don't have a sub-status field there is no way to capture the intended disposition of acceptance/rejection when a CAP is in Final Comment Period.

Should we change the workflow so that rejected proposals go from Awaiting Decision to Rejected instead of parking in FCP for a week?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's important to give poor disposition proposals the same week to volley final arguments as popular ones. That way, we have a full history of why something failed, and we can move on from it with confidence.

I also think we should add a line explaining the edge case of moving from FCP to Draft, which has historically been around blocking changes. Like: "...back into a Draft state, if, for instance, implementation depends on other new CAPs."

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another option would be to add a Notes (or Comments or Remarks) field to the CAPs that can capture the intended disposition of acceptance or rejection for CAPs going from Awaiting Decision to Final Comment Period. This would help in both cases,

  1. Capturing the rejection reason when CAPs are in FCP but headed towards Rejected, and
  2. Capturing comments when moving from FCP to Draft.

Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants