Feature Request
Describe the problem you need a feature to resolve.
The problem here is that instead of the command scorecard and bundle has a standard to return the result with a list of errors we have a different implementation to achieve the same for each command which reduces the maintainability as make it harder for who will consume the result via a pipeline for example.
See that the bundle validate uses the following implementation to return a list of errors in the project:
result = NewResult()
result.AddError(errors.New(string.fmt(("operator deployment %q not available: %s", ds.Name, s.Reason))))Then, another subcommand of the same bundle command, ( bundle run ), has another implementation done in the PR: #3908 to achieve the same.
Describe the solution you'd like.
So, IMO we should move the result implementation for internal/util and keep it a standard to output a list of errors.
Feature Request
Describe the problem you need a feature to resolve.
The problem here is that instead of the command scorecard and bundle has a standard to return the result with a list of errors we have a different implementation to achieve the same for each command which reduces the maintainability as make it harder for who will consume the result via a pipeline for example.
See that the bundle validate uses the following implementation to return a list of errors in the project:
Then, another subcommand of the same bundle command, ( bundle run ), has another implementation done in the PR: #3908 to achieve the same.
Describe the solution you'd like.
So, IMO we should move the result implementation for internal/util and keep it a standard to output a list of errors.