-
Notifications
You must be signed in to change notification settings - Fork 83
Update the dependency installation process to capture and log output #2226
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
base: main
Are you sure you want to change the base?
Conversation
…from pip command to enable debugging.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2226 +/- ##
==========================================
- Coverage 64.05% 64.01% -0.04%
==========================================
Files 100 100
Lines 8624 8629 +5
Branches 893 895 +2
==========================================
Hits 5524 5524
- Misses 2928 2933 +5
Partials 172 172 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
✅ 129/129 passed, 9 flaky, 5 skipped, 10m22s total Flaky tests:
Running from acceptance #3438 |
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.
@eri-adepoju, can this only be part of the debug log level, not during regular processing?
|
@sundarshankar89 I've edited it to be more verbose at the debug level. |
gueniai
left a comment
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.
LGTM
Changes
Linked issues
Resolves #2225
Functionality
runcommand to usecapture_output=Trueandtext=Truefor better output handling.Tests