Skip to content

PEP 708: Mark as Rejected#4922

Open
pfmoore wants to merge 3 commits intopython:mainfrom
pfmoore:reject_708
Open

PEP 708: Mark as Rejected#4922
pfmoore wants to merge 3 commits intopython:mainfrom
pfmoore:reject_708

Conversation

@pfmoore
Copy link
Copy Markdown
Member

@pfmoore pfmoore commented Apr 16, 2026


📚 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
Copy link
Copy Markdown
Member Author

pfmoore commented Apr 16, 2026

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
Copy link
Copy Markdown
Member

hugovk commented Apr 16, 2026

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
Copy link
Copy Markdown
Member Author

pfmoore commented Apr 16, 2026

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.

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.

2 participants