Skip to content

cmd/go: go test flag -failfast should be cacheable #47355

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.

Sign up for GitHub

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

Jump to bottom
Closed
anacrolix opened this issue Jul 23, 2021 · 2 comments
Closed

cmd/go: go test flag -failfast should be cacheable #47355

anacrolix opened this issue Jul 23, 2021 · 2 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Backlog

Comments

@anacrolix
Copy link
Contributor

Running tests with -failfast should use the test cache. It was unexpected that this was not the case. #42544 uncovers that this is documented. My use case is running go test all in projects with a large number of dependencies and reporting failing tests to them.
@ianlancetaylor ianlancetaylor changed the title go test flag -failfast should be cacheable cmd/go: go test flag -failfast should be cacheable Jul 23, 2021
@ianlancetaylor
Copy link
Contributor

CC @bcmills @jayconrod

@jayconrod jayconrod added the NeedsFix The path to resolution is known, but the work has not been done. label Jul 23, 2021
@jayconrod jayconrod added this to the Backlog milestone Jul 23, 2021
@korzhao korzhao mentioned this issue Jul 24, 2021
Closed
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/337229 mentions this issue: cmd/go: go test flag -failfast should be cacheable

@golang golang locked and limited conversation to collaborators Aug 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

cmd/go: go test flag -failfast should be cacheable
4 participants
@jayconrod @anacrolix @ianlancetaylor @gopherbot