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

Feat: add DisableUnexported #122

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

flw-cn
Copy link

@flw-cn flw-cn commented Feb 4, 2020

DisableUnexported specifies whether to disable the unexported fields of struct.
This is useful for debugging APIs.

This should close #121.

DisableUnexported specifies whether to disable the unexported fields of struct.
This is useful for debugging APIs.
@flw-cn
Copy link
Author

flw-cn commented Jun 24, 2020

I noticed that the previous changes did not have accompanying test cases, which resulted in a slight drop in test case coverage. For that reason, I've dedicated today to writing a set of test cases to show that I'm doing it right.

There are still some errors in the CI process, but I don't think they're caused by my changes.

@davecgh Could you take a look at this PR at a convenient time?

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.

Request to add features that can suppress unexported fields
1 participant