-
Notifications
You must be signed in to change notification settings - Fork 2
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 README #80
Update README #80
Conversation
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.
Thanks for updating this README! It looks really great. I have two suggestions :)
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.
Thanks for updating the README! I approve of this doc change 👍
(Not giving an official approval since there are code changes in this PR.)
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.
A few more small comments related to the style guide
Co-authored-by: Erin Conley <[email protected]>
Co-authored-by: Erin Conley <[email protected]>
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.
Thanks for updating the README! I approve of the doc changes 👍
(Not giving an official approve due to the code changes in this PR.)
Test coverage for 7b2131d
Static code analysis report
|
Overview
Updated the README file to be compliant with is-charms-template-repo
Checklist
src-docs
urgent
,trivial
,complex
)