The existing code wasn't always checking errors and freeing resources correctly. It's quite difficult, especially to handle all error paths correctly.
So instead of "handcrafting" command pipelines using low-level facilities like exec.Cmd, switch to using the pipe package that was introduced in #90. That package takes care of most of the details internally, presenting a more convenient and less error-prone UI.
This change clears up a bunch of linter errors regarding unchecked errors.
Note that this PR branch is built on top of the changes that were added in PRs #89 and #90. Those PRs can be merged separately or in combination with this one.
/cc @tgummerer and @ttaylorr as possible reviewers
The existing code wasn't always checking errors and freeing resources correctly. It's quite difficult, especially to handle all error paths correctly.
So instead of "handcrafting" command pipelines using low-level facilities like
exec.Cmd
, switch to using thepipe
package that was introduced in #90. That package takes care of most of the details internally, presenting a more convenient and less error-prone UI.This change clears up a bunch of linter errors regarding unchecked errors.
Note that this PR branch is built on top of the changes that were added in PRs #89 and #90. Those PRs can be merged separately or in combination with this one.
/cc @tgummerer and @ttaylorr as possible reviewers