-
Notifications
You must be signed in to change notification settings - Fork 162
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 testing info do DEVELOPER_GUIDE.md #1554
Adding testing info do DEVELOPER_GUIDE.md #1554
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1554 +/- ##
=======================================
Coverage 66.06% 66.06%
=======================================
Files 93 93
Lines 2328 2328
Branches 310 310
=======================================
Hits 1538 1538
Misses 722 722
Partials 68 68 |
Signed-off-by: Miki <[email protected]> Co-authored-by: Craig Perkins <[email protected]> Co-authored-by: Ryan Liang <[email protected]> Signed-off-by: Sam <[email protected]>
Signed-off-by: Sam <[email protected]>
6e65a7b
to
c90d79a
Compare
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 @samuelcostae! This looks good to me. Left 2 small comments.
Signed-off-by: Sam <[email protected]>
Added the suggested changes |
@scrawfor99 @peternied Could one of you also review this PR? |
Description
Adding details to testing section of developer_guide.br , referencing the OS Dashboards testing project
Category
Documentation
Why these changes are required?
Missing information on Docs.
What is the old behavior before changes and new behavior after changes?
Issues Resolved
Testing
Manual Review
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.