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
feat: propose proposal template #7
Conversation
- add a template for future Strimzi proposals, based on existing proposals Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally I like the basic proposal template offered here.
I suggest this be used as an optional template for proposals if someone wants to use this.
I.e. I don't think a proposal has to use this template to avoid unneeded process.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My view ...
- I'm probably missing some _Current situation section to describe the current state as a starting point.
- I think also explicit section about Backwards compatibility might be useful as a reminder.
- On the other hand I think the split of the Proposal section goes maybe a bit too deep. I would probably remove the subchapters.
Overall - as mentioned by Sam - I would make it clear this is optional and I don't have a big issue with this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no more comments than the Jakub's ones and I agree that we should make it clear that the template is optional.
- Add sections for compatibility and the current situation - Add a comment to the proposal mentioning this is for example purposes and that it can be followed/modified to best suit the proposal being made Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
Hi All, thank you for the comments. I have just updated the PR to reflect them. |
@Frawless @tombentley any comments on this? |
Thanks for the PR @matthew-chirgwin |
proposals
Signed-off-by: Matthew Chirgwin chirmatt@uk.ibm.com