Skip to content
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

Reporter should export unexported fields #180

Open
dsnet opened this issue Jan 3, 2020 · 0 comments
Open

Reporter should export unexported fields #180

dsnet opened this issue Jan 3, 2020 · 0 comments

Comments

@dsnet
Copy link
Member

@dsnet dsnet commented Jan 3, 2020

The main functionality of cmp does not automatically handle unexported fields to signal to the user that they should extra thought should be given to how the type should be compared.

This rationale does not apply for the reporter which is only for human debugging. The reporter should (on a best-effort basis) try to export unexported fields so that it can call the String method if available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.