-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
create a recipe template #9841
create a recipe template #9841
Conversation
this can help new contributors, and reviewers too
This comment has been minimized.
This comment has been minimized.
It would be very nice if combined with the |
@SpaceIm maybe you have some sort of checklist you use for modernization of recipes ? Can it somehow be translated into this template ? |
Somewhere in my head yes. I'll take a look. |
I also had some templates, but didn't update them for a while: https://github.com/SSE4/cci.templates |
I'll do a pr on your repo then. It's more usable as a proper template. I did not know not know that |
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.
Excellent initiative!
yes, why not. I have been thinking how to properly organize that, but probably forgot about it at some point in time. |
Co-authored-by: Uilian Ries <[email protected]>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Stella Smith <[email protected]>
This comment has been minimized.
This comment has been minimized.
Co-authored-by: SpaceIm <[email protected]>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
2 similar comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Let's try to move this forward! At least this seems like a good starting point
Updating docs! |
this can help new contributors, and reviewers too.
This is a very quick draft.
We should probably also add more comments to this template, in order to explain all these conventions.
Also, if you find this template could be useful, then we should link to it from the documentation .md files
I added all CCI's approved reviewers as contributor to my fork, so please edit the files directly, instead of doing code suggestion.
this partially answers item 3. in #2857 (comment)