GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||
GeezerVison is showing github.com — Infra: only create peps.json at /api/peps.json by hugovk · Pull Request #4231 · python/peps Conversation
|
|
'Fixed a bug by removing the feature', you might say! I think the converse is true here, A |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
:) Well, this bug/feature was added in a large-ish refactoring PR (#3417) which made no mention of the addition in the description, and was merged 10 minutes later with no review. I only realised this duplicate endpoint existed when #4226 was opened yesterday, and would have objected to the addition at the time, as we had a long discussion at #2584 about whether to move it to the root, and in the end decided not to.
I see it as a simple read-only JSON API, much like https://endoflife.date/docs/api, https://swapi.dev and https://open-meteo.com.
I think it can cause confusion to have two copies. Are they different? Will one go away? If we change one, will the other also change? If we start versioning it, do we need to add another duplicate to the root? Or if we expose JSON for each individual PEP, do they all go to the root? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Only create peps.json at /api/peps.json
Only create peps.json at /api/peps.json
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.