-
Notifications
You must be signed in to change notification settings - Fork 718
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
Non-exported variables shouldn't be documented... #150
Comments
Ok sorry, now I see the checkbox in the UI. But I still think it's weird that it's off by default or even that it's an option. Can we have an option not to have the option? :P |
Ok I found it :( |
Great, an issue that has resolved itself 😉 |
This behavior still is a bit weird. Because of the checkbox at the top. |
You are right, the checkbox makes no sense if the parameter is specified, I've adjusted the template. |
It's getting confusing seeing my local/internal variables/constants/functions documented. :(
These shouldn't be exposed to the consumer, or at least there should be a switch to turn these off.
Personally, they should not be exposed by default. Only what get's exported should be documented.
The text was updated successfully, but these errors were encountered: