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

Test omitempty for pointers #370

Closed

joukewitteveen wants to merge 1 commit into BurntSushi:master from joukewitteveen:omitpointer

joukewitteveen commented Oct 23, 2022

The value behind the pointer should not matter, just whether the pointer is nil or not.


          Test omitempty for pointers

9574faf

The value behind the pointer should not matter, just whether the pointer
is nil or not.

joukewitteveen mentioned this pull request

omitempty incorrectly looks at values behind pointers #371

Closed

Collaborator

arp242 commented Oct 24, 2022

I added a more expansive test for this (with a t.Skip() for now) in 68ce1fc.

arp242 closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment