Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OEP-37 Dev Data: Evaluate whether to carry this forward or not. #607

Closed
Tracked by #573
sarina opened this issue Jul 5, 2024 · 4 comments · Fixed by #674
Closed
Tracked by #573

OEP-37 Dev Data: Evaluate whether to carry this forward or not. #607

sarina opened this issue Jul 5, 2024 · 4 comments · Fixed by #674
Labels
feb25 help wanted Ready to be picked up by anyone in the community

Comments

@sarina
Copy link
Contributor

sarina commented Jul 5, 2024

Several OEPs were authored aspirationally, and then never implemented. Other were partially implemented, but have stalled or diverged from reality.

In order to establish OEPs as a trustworthy source of documentation, we should mark OEPs that never came to be as "Replaced" or "Obsolete" (whichever or is more appropriate). We should amend OEPs that are still relevant but mismatched with reality.

OEP-37 is one of these, as called out in #573

@sarina sarina added the help wanted Ready to be picked up by anyone in the community label Jul 5, 2024
@sarina
Copy link
Contributor Author

sarina commented Feb 2, 2025

@feanil or @kdmccormick do you have any opinions on this? I notice the OEP is specific to Devstack -- perhaps this should simply be obsoleted, and let TEPs cover dev data for Tutor?

@sarina sarina added the feb25 label Feb 2, 2025
@kdmccormick
Copy link
Member

The OEP mentions devstack a few times, but the implementation is not devstack-specific. Those references could be removed or replaced with Tutor without significantly altering the OEP's content.

I think its current status ("Deferred") is appropriate, but I think it would be good if it that status was more obvious. For example, rather than being listed alongside the Provisional and Accepted Best Practice OEPs, maybe "Deferred" OEPs should get kicked down to the bottom section on the sidebar along with Obsolete and Replaced?

Image

@sarina
Copy link
Contributor Author

sarina commented Feb 3, 2025

I see two options - it's not really Obsolete or Replaced so if we put in that section I don't think it'll ever get implemented. But, I see the point that "Deferred" could basically be read as "Obsolete".

Another could be to put Deferred in the title of the OEP itself so it would look like

Image

I feel that's a bit clearer that we like the OEP, it's not specifically Obsolete, but its implementation hasn't been done yet.

@kdmccormick
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feb25 help wanted Ready to be picked up by anyone in the community
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants