-
Notifications
You must be signed in to change notification settings - Fork 181
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
Adding the COMMUNICATION.md to the Standard Base Documentation pattern #589
Conversation
Added COMMUNICATION.md and ADI instance
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.
@kschueths thank you for taking the time to contribute this extension to the pattern!
I fixed a couple of formatting/wording changes myself.
Also made a change so that the COMMUNICATION.md will be rendered in the online book correctly.
Also left some inline comments. Merged one of them already that I was pretty certain of :)
If you could address the remaining comments, then I can get this merged in the very near future.
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.
Thank you for catching those
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.
This looks good now.
Will wait for checks to pass and then merge.
Thanks again for sharing your COMMUNICATION.md approach with us @kschueths. |
InnerSourceCommons#589) * Added COMMUNICATION.md and ADI instance to base-documentation.md. * Adding COMMUNICATION template to the sidebar of the book (en only)0
Added COMMUNICATION.md and ADI instance