Skip to content
This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Add the ability to suppress non-zero exits as errors #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dnnrly
Copy link

@dnnrly dnnrly commented May 10, 2022

This allows you to test the output exactly as returned by the binary, without having to parse the error to extract any meaning.

@dnnrly dnnrly requested a review from a team as a code owner May 10, 2022 22:47
@MuweiHe
Copy link

MuweiHe commented Jan 17, 2023

LGTM!

@brianstrauch
Copy link
Member

The latest changes from master will have to be merged in for CI to pass @dnnrly

@dnnrly
Copy link
Author

dnnrly commented Jan 23, 2023

Apologies, I'd completely forgotten I'd raised this PR. I'm afraid I'm away from my laptop for a little while so won't be able to fix anything if it breaks when I merge those changes in.

Given the upcoming changes in 1.20, does this PR still add value?

@brianstrauch
Copy link
Member

Given the upcoming changes in 1.20

Not that I'm aware of, what specific changes are you thinking of?

@dnnrly
Copy link
Author

dnnrly commented Jan 23, 2023

I was thinking of the ability to create a version of your build with coverage profiling:
https://go.dev/testing/coverage/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants