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

Update SEP statuses #1593

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

chowbao
Copy link

@chowbao chowbao commented Dec 12, 2024

  • Update SEP readme with the current statuses of SEPs committed in the repo
  • New status names
    • Active --> Final
    • Rejected --> Abandoned

@chowbao
Copy link
Author

chowbao commented Dec 12, 2024

TODO: will update the old SEPs in draft as I read through the threads and see if they should be revived or abandoned

  • Remove Active --> Final
  • Separate Rejected --> Abandoned changes into separate PR (update readme)

@leighmcculloch leighmcculloch marked this pull request as draft December 12, 2024 22:55
@leighmcculloch
Copy link
Member

leighmcculloch commented Dec 12, 2024

If rejected is being replaced with abandoned (which I support), the README docs describing the statuses should be updated. Fwiw I think it's worth doing that change alone in a separate PR, so that the what and why are hyperfocused on one decision. One decision, one PR, is easier for folks to track why something is happening, and debate it in isolation of other decisions.

Copy link

This pull request is stale because it has been open for 30 days with no activity. It will be closed in 30 days unless the stale label is removed.

@github-actions github-actions bot added the stale label Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants