alexellis / alexellis/go-execute

execErr seems to be ignored other than for exit code

オープン
#20 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Go
スター
177
フォーク
10
PR マージ指標
30日以内にマージされた PR はありません

説明

## Expected Behaviour

I would expect us to do _something_ with `execErr` [here](https://github.com/alexellis/go-execute/blob/master/exec.go#L180-L185) unless it can truly be ignored, but its not clear if it can be.

## Current Behaviour

`execErr` is dropped on the floor if its not an exit error.

## Possible Solution

Return it if `ctx.Err()` is nil?

## Steps to Reproduce (for bugs)

N/A

## Context

Just curious as to whether ignoring it is the correct thing to do.

## Your Environment

* `go version` : go version go1.21.3 darwin/arm64

* Operating System and version (e.g. Linux, Windows, MacOS): MacOS Ventura 13.6

* Link to your project or a code example to reproduce issue: N/A

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。