Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@TimWolla
Copy link
Member

@TimWolla TimWolla commented Aug 29, 2025

@TimWolla TimWolla changed the title Extend discussion and voting period rules Clarify discussion and voting period rules Aug 29, 2025
@TimWolla TimWolla force-pushed the discussion-and-voting-period branch from e267afb to 22cc4b3 Compare August 29, 2025 19:19
The actual start of the vote MUST be announced on the mailing list in a separate
thread with a ``[VOTE]`` prefix followed by the RFC title as the Subject. The
email MUST include a link to the Wiki page of the RFC and to the mailing list
archives of the discussion thread. It furthermore MUST clearly specify the

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Since the update to include the full threading view, https://news-web.php.net/php.internals would be the preferred link. But I would also be good with externals, since the IDs match up and folks are able to use either one. What's important to me is that folks are able to easily find the discussion.

Copy link
Member

@iluuu1994 iluuu1994 left a comment

Choose a reason for hiding this comment

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

I realize some of these should go on the list. But I'm late in the discussion (barely working due to personal reasons). None are significant enough for me to object, so discard if you like, or ask me to post publicly.

If the proposer is not a member of php.net and thus can not create RFCs on the
wiki, they should recruit one of the members for help or request membership.
Prior to starting a vote, RFC authors MUST post an Intent to Vote message to the
discussion thread. The post MUST be made at least 2 days and no more than 7 days
Copy link
Member

Choose a reason for hiding this comment

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

IMO, the upper bound (at least such a low one) is pretty unnecessary. As long as nothing has changed, internals have effectively already soft-approved the vote by not objecting.

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the “Intent to Vote” becomes meaningless if folks don't follow up on it. To me it's not just a “last call for change requests”, but also moving the RFC discussion to the top of the INBOX to make it easier for folks to find it when the vote actually happens and as a clear communication that the RFC authors consider the RFC finished. It would be odd for them to leave the RFC sitting around for another month before then actually starting the vote.

Copy link
Member

@iluuu1994 iluuu1994 Oct 6, 2025

Choose a reason for hiding this comment

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

Saying "I'll start the vote soon", and doing so in 10 days doesn't seem problematic to me, especially for very complex and drawn-out RFCs. Actually, 2 days may not even be enough for people to have the time to re-read large RFCs. Given that's effectively the aim of this reminder, it might make sense to be a bit more lenient.

Edit: Okay, we should use the upper bound for this argument, i.e. the 7 days. But for very complex RFCs, 7 days may still not be enough for peopel with busy schedules.


After the voting period has started, including after the vote closed and the RFC
is either accepted or declined, there MUST NOT be any further Major or Minor
changes to the RFC text and making editorial changes SHOULD be avoided for the
Copy link
Member

@iluuu1994 iluuu1994 Oct 5, 2025

Choose a reason for hiding this comment

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

What about mistakes in the examples? I think we fixed some examples months after RFCs have been accepted. Keeping the mistakes would be confusing. Obviously, this assumed no malicious intent (e.g. fixing obvious mistakes, not tweaking semantics or sneaking in new features).

Edit: I see this is referring only to the voting period, whereas I interpreted it as "from then on". Maybe adjust to "during the voting period"? Still doesn't explain what should happen if a mistake is spotted in an example during the vote.

Edit 2: Nevermind. It says "including after the vote closed and the RFC is either accepted or declined", which I overlooked on my second read. This seems to be in conflict with the errata section.

Copy link
Member Author

Choose a reason for hiding this comment

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

This seems to be in conflict with the errata section.

The errata section is intended “append-only”. The RFC remains as a record of what was being voted on and the errata section explains why that didn't happen.

Still doesn't explain what should happen if a mistake is spotted in an example during the vote.

If it's an obvious mistake it falls under the typo rule. If the example is actively misleading with regard to semantics, this might have an impact on the vote (folks might vote differently had the example been differently). In that case, this should be resolved as any other mistake with significance (e.g. by prominently mentioning it on the list and if necessary canceling the vote).

Copy link
Member

Choose a reason for hiding this comment

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

The errata section is intended “append-only”. The RFC remains as a record of what was being voted on and the errata section explains why that didn't happen.

I still don't find this good, because frequently people look only at examples and expect them to be correct.

If it's an obvious mistake it falls under the typo rule.

Reality is a bit more nuanced. Sometimes there are semantic mistakes, but maybe not the ones that the example is highlighting. I don't think such an example should be grounds for invalidating a vote, when the semantics are sufficiently clarified in the RFC text and other examples.

In that case, this should be resolved as any other mistake with significance (e.g. by prominently mentioning it on the list and if necessary canceling the vote).

As mentioned, this also happens for RFCs that have long passed and even been merged.

avoidance of doubt. The voting details (such as the voting period or
interpretation of secondary votes) MUST NOT be changed after the vote opened.

The voting period MAY be canceled within the first 2 days in case of severe
Copy link
Member

Choose a reason for hiding this comment

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

What's the rationale of restricting the cancling of the vote to the first two days? If somebody were to point out a major problem later in the voting period, wouldn't it make sense for the authors to invalidate the vote and solve the issue, rather than the RFC being accepted in a flawed state? Of course, they could still choose to simply not merge it, and create a follow up RFC that solves the issue, but then there's no clear path forward if that second vote fails.

Copy link
Member Author

Choose a reason for hiding this comment

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

see https://externals.io/message/128594#128724: This is to prevent folks from canceling the vote at 13 days and 23 hours if they are unhappy with the results. At a certain point it's better to just let the vote run its course.

In case of “honest mistakes” with the RFC, I expect follow-up RFCs to be well-received by the community. And if there's an obvious solution to the mistake, this can also be solved by “simple agreement” using the Errata mechanism.

Copy link
Contributor

@theodorejb theodorejb left a comment

Choose a reason for hiding this comment

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

I noticed a couple grammar mistakes.

as described in the "Required Majority" section below.

To account for minor calculation errors due to timezone changes, outside
commitments by RFC authors, and others deadlines, RFC authors MAY use a grace
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
commitments by RFC authors, and others deadlines, RFC authors MAY use a grace
commitments by RFC authors, and other deadlines, RFC authors MAY use a grace

Copy link
Member Author

Choose a reason for hiding this comment

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

The RFC is intentionally voting on a specific commit hash to clearly link RFC and PR. For the avoidance of doubt, I'm therefore not even fixing typos at this point.

Can you please send them as a separate follow-up PR if / when the RFC gets accepted and this PR merged?


Due to the significance of the end-of-year holidays for a majority of the world,
the voting period MUST NOT start and MUST NOT end in the period between
December, 17th 00:00 UTC and January, 10th 00:00 UTC.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
December, 17th 00:00 UTC and January, 10th 00:00 UTC.
December 17th, 00:00 UTC and January 10th, 00:00 UTC.

@TimWolla TimWolla marked this pull request as ready for review November 20, 2025 09:33
@bukka bukka merged commit 59430c3 into php:main Nov 20, 2025
1 check passed
@TimWolla TimWolla deleted the discussion-and-voting-period branch November 20, 2025 10:02
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.

6 participants