Skip to content

Conversation

@ccoVeille
Copy link
Collaborator

Summary

vendor go-spew

Changes

Import unmaintained library

Motivation

go-spew is unfortunately unmaintained, we cannot expect a fix any time soon.

vendoring go-spew, the same way we vendored go-difflib would help us to remove bugs, but also keep the current pretty print lib without moving to something else.

Note: we have already vendored the go-difflib we are using to compare the go-spew results.

Related issues

Closes #1826

Copy link
Collaborator

@brackendawson brackendawson left a comment

Choose a reason for hiding this comment

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

The license is handled properly and is compatible with what we're doing. I have checked and the import is a faithful verbatim copy. In this case we didn't keep the history as we did with difflib, but we decided that's not mandatory in the vendoring of ernesto-jimenez/gogen. The modifications to fit it into the internal directory are also fine.

Big!

@brackendawson brackendawson merged commit 65697ce into stretchr:master Nov 28, 2025
9 checks passed
@ccoVeille ccoVeille deleted the vendor-gospew branch November 28, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

go-spew vendoring

2 participants