Skip to content

PEP 708: Mark as Rejected#4922

Merged
pfmoore merged 3 commits into
python:mainfrom
pfmoore:reject_708
Apr 17, 2026
Merged

PEP 708: Mark as Rejected#4922
pfmoore merged 3 commits into
python:mainfrom
pfmoore:reject_708

Conversation

@pfmoore

@pfmoore pfmoore commented Apr 16, 2026

Copy link
Copy Markdown
Member

📚 Documentation preview 📚: https://pep-previews--4922.org.readthedocs.build/

@pfmoore pfmoore requested a review from dstufft as a code owner April 16, 2026 20:57
@pfmoore

pfmoore commented Apr 16, 2026

Copy link
Copy Markdown
Member Author

How do I have two resolution headers? Separating the URLs with commas didn't work, nor did having two Resolution: lines. For a provisionally accepted, and then rejected, PEP I would like to keep a record of both decision posts, for the record.

Worst case scenario, I could move the URL for the provisional acceptance into the text of the rejection note, but that doesn't feel right.

Comment thread peps/pep-0708.rst Outdated
@hugovk

hugovk commented Apr 16, 2026

Copy link
Copy Markdown
Member

How do I have two resolution headers? Separating the URLs with commas didn't work, nor did having two Resolution: lines. For a provisionally accepted, and then rejected, PEP I would like to keep a record of both decision posts, for the record.

Worst case scenario, I could move the URL for the provisional acceptance into the text of the rejection note, but that doesn't feel right.

We'd need to adjust the code to allow two resolutions.

I'd just point to the most recent one, we have the post history for older things.

@hugovk hugovk changed the title Reject PEP 708 PEP 708: Mark as Rejected Apr 16, 2026
@pfmoore

pfmoore commented Apr 16, 2026

Copy link
Copy Markdown
Member Author

I'd just point to the most recent one, we have the post history for older things.

OK, it's not ideal IMO, but it'll do.

@pfmoore pfmoore merged commit b0b1f2e into python:main Apr 17, 2026
5 checks passed
@pfmoore pfmoore deleted the reject_708 branch April 17, 2026 09:54
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.

3 participants