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

add proposal document template #1449

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gregsdennis
Copy link
Member

@gregsdennis gregsdennis commented Sep 24, 2023

Resolves #1443

I started working on the propertyDependencies extraction and the template kinda just worked itself out. It turns out leaving questions is easier that actually filling out the sections.

@gregsdennis gregsdennis added this to the stable-release milestone Sep 24, 2023
@gregsdennis
Copy link
Member Author

See ☝️ for an example of this template in use.

@gregsdennis gregsdennis self-assigned this Sep 30, 2023
@gregsdennis
Copy link
Member Author

We're having some debate on the purpose of this document over on #1450. Holding this until that resolves.

@Relequestual Relequestual self-requested a review October 12, 2023 09:52
@Relequestual
Copy link
Member

Based on the discussion and activity at #1450, I feel this PR needs a suplementary document which explains the purpose of the proposal document (or documents if that turns out to be the case).

I would think such discussion needs to be had at #1443 and agreed on before making further changes here.

I'm happy to facilitate these discussions, either in text form or via calls. @gregsdennis @jdesrosiers let me know which you would prefer. (Anyone else is also welcome to be involved, but Greg and Jason are the primary individuals driving this right now.)

Comment on lines +18 to +26
## Status

**Current Status**: PROPOSAL

<!-- TODO: We should have a short standard blurb outlining the stages involved in a
feature making its way to stable status. -->

<!-- TODO: Link to a document that describes the proposal => stable process in
detail. -->
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Starting to think this can go away completely. The document is a proposal.

Maybe we want to keep it around for posterity, but I think we could use file paths to handle that. Maybe something like moving the files to a completed/<release>/ folder and renaming the file to <filename>-rejected.md / <filename>-accepted.md.

Personally, I'm happy with just deleting the file once we don't need it anymore, whether it was rejected or accepted.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be a good idea to use the status field to communicate the maturity of the proposal. For example, something like,

  1. Development -- The proposal is in the early stages of being considered
  2. Release Candidate -- No further changes are expected, but we want to see as much implementation and usage to flush out any remaining issues
  3. Awaiting Release -- The proposal has been excepted and will be included in the next release. There will be no more changes.
  4. Released
  5. Rejected

We don't even need to have a fixed list of statuses we're limited to. I think what's important is that it communicate something useful.

As for whether to keep around completed proposals, I think it's best to keep them around. Technically, they're never gone because they're in the git history, but people aren't going to go digging through our git history to find a proposal they're watching. I'm imagining someone trying to find the status of a proposal they're interested in and being confused because they can't find it or any explanation why it disappeared. For the same reason, I think it's best not to move them to a "rejected" or "completed" folder because that would break any bookmarks people have to those proposals. They can maybe be removed after some generous period of time, but not right away.

Comment on lines +60 to +62
### Alternatives

<!-- What other options have been considered? (summary, not detailed) -->
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per #1443 (comment), maybe this should go. This would be important for an ADR, but it's not necessarily something that the public needs to see.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still not entirely sure where you want to draw the line between the two documents, but I agree that a discussion of the alternatives isn't necessary in this document except maybe as an appendix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Should we have a supplementary spec template for proposals?
3 participants