-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore(doc): Update README #81
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 the README! Only one comment from me :)
Co-authored-by: Erin Conley <erin.conley@canonical.com>
…-provider into update-readme
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 approval as .trivyignore
was changed 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.
Thank you! Some comments.
Test coverage for f9ecf91
Static code analysis report
|
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! The updated version looks good to me, I approve 👍
(Not giving an official approval since .trivyignore
was changed in this PR.)
Applicable spec:
Overview
Rationale
Module Changes
Checklist
urgent
,trivial
,complex
)