CCBR / Tools

Utilities for CCBR Bioinformatics Software ⚒️
https://ccbr.github.io/Tools/
MIT License
1 stars 4 forks source link

fix: don't add extra newline to command stdout/stderr #10

Closed kelly-sovacool closed 1 month ago

kelly-sovacool commented 1 month ago

Changes

shell_run() and exec_in_context() were adding extra newlines to command output, which messed up other functions that used them

Issues

NA

PR Checklist

(~Strikethrough~ any points that are not applicable.)

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 35.37%. Comparing base (8de92ce) to head (d8af76e). Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/ccbr_tools/shell.py 83.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10 +/- ## ========================================== + Coverage 35.22% 35.37% +0.15% ========================================== Files 13 13 Lines 1042 1046 +4 ========================================== + Hits 367 370 +3 - Misses 675 676 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.