cli icon indicating copy to clipboard operation
cli copied to clipboard

Remove unnecessary process wait on MacOS (issue #1523)

Open cedel1 opened this issue 1 year ago • 2 comments

Remove process.communicate() in MacOS causing wait for subprocess even when the subprocess results were not used.

cedel1 avatar Aug 31 '23 15:08 cedel1

Hmm, I am surprised by the unrelated codecov check failure (especially since that part wasn't changed for quite some time, it fails on master and tests in previous PRs passed. Version changes?

cedel1 avatar Aug 31 '23 17:08 cedel1

Codecov Report

Patch coverage is 89.91% of modified lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

Files Changed Coverage
httpie/output/ui/man_pages.py 0.00%
httpie/output/ui/rich_utils.py 0.00%
tests/test_binary.py ø
tests/test_defaults.py ø
tests/test_output.py ø
tests/test_parser_schema.py ø
tests/test_plugins_cli.py ø
tests/test_redirects.py ø
tests/test_regressions.py ø
tests/test_sessions.py ø
... and 90 more

:loudspeaker: Thoughts on this report? Let us know!.

codecov-commenter avatar Aug 31 '23 23:08 codecov-commenter