Skip to content

Conversation

ivanvc
Copy link
Member

@ivanvc ivanvc commented Apr 4, 2024

With the update to Go 1.22.2, this workaround is no longer needed.

This reverts commit da7ab15.

Fixes #17560.

Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.

@ivanvc ivanvc changed the title Revert "tests: temporary disable coverage redisign Go experiment" Revert "tests: temporary disable coverage redesign Go experiment" Apr 4, 2024
With the update to Go 1.22.2, this workaround is no longer needed.

This reverts commit da7ab15.

Signed-off-by: Ivan Valdes <ivan@vald.es>
@ivanvc ivanvc force-pushed the remove-go-1-22-1-coverage-workaround branch from 9e3c7c4 to 5cd14a6 Compare April 4, 2024 18:17
Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@ahrtr ahrtr merged commit 9cb33de into etcd-io:main Apr 5, 2024
@ivanvc ivanvc deleted the remove-go-1-22-1-coverage-workaround branch April 5, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Update to Go v1.22 broke getting test coverage
3 participants