-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: build command #24
Conversation
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
This allows `go mod tidy` to run successfully while keeping Gazelle happy. Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
SilenceUsage: true, | ||
SilenceErrors: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alexeagle I'm not entirely sure if we want this here at the root cmd or not. I'd be tempted to leave it in the root for now and see how other commands interact with it. Just something to be aware of.
cc @mattem
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
Not licensed for re-use. | ||
*/ | ||
|
||
package aspecterrors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't we have to change the Bazel Spawner interface to return this error type?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. The ExitError satisfies the default error interface. The change I made to the main func checks whether the error contains the exit code or not. Also, we cannot change the signature of the Run method since it has to match the signature defined by cobra for the RunE field in the Command struct.
Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
Description
Introduces the
aspect build
command that performs a simplebazel build
without any flags.Test plan
aspect build //...
.