-
Notifications
You must be signed in to change notification settings - Fork 10
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
Set default log mode to Warn+ and streamline error handling and cleanup #243
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
Test Results 4 files ±0 48 suites +4 5s ⏱️ -2s Results for commit 90907b7. ± Comparison against base commit 4cced48. This pull request removes 9 and adds 8 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Code Climate has analyzed commit 90907b7 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 60.5% (50% is the threshold). This pull request will bring the total coverage in the repository to 78.6% (-1.1% change). View more on Code Climate. |
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.
My review comments were addressed.
This update includes the following changes:
error cbuild: exit status 1
.csolution.yml
is now the expected extension and.csolution.yaml
is no longer acceptedNote: Please be informed that to get the detailed logs use
--verbose
option