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

[DOC beta] Hide private classes from YUIDoc #4052

Merged
merged 1 commit into from
Jan 9, 2016
Merged

[DOC beta] Hide private classes from YUIDoc #4052

merged 1 commit into from
Jan 9, 2016

Conversation

pangratz
Copy link
Member

@pangratz pangratz commented Jan 7, 2016

It looks like marking classes as @private is not enough to hide them
from showing up in the API doc at http://emberjs.com/api/data/.

This commit changes the used comments to start with /*, which are
definitely ignored by JSDoc.

@rwjblue
Copy link
Member

rwjblue commented Jan 7, 2016

👍

@rwjblue
Copy link
Member

rwjblue commented Jan 7, 2016

(we use YUIDoc, but JSDoc and YUIDoc agree here)

@fivetanley
Copy link
Member

Seems like the website should filter out private classes? They are still marked as private on the website.

@rwjblue
Copy link
Member

rwjblue commented Jan 7, 2016

@fivetanley - yes, https://github.com/emberjs/website/blob/master/lib/api_docs.rb should be better about not including private classes in the sidebar

@pangratz pangratz changed the title [DOC beta] Hide private classes from JSDoc [DOC beta] Hide private classes from YUIDoc Jan 8, 2016
@bmac
Copy link
Member

bmac commented Jan 9, 2016

@pangratz do you mind also removing the 2nd * from the method definitions (createSnapshot ect) in internal-model.js? iirc if yui doc doesn't see a class for these it will attribute these methods to the wrong class.

It looks like marking classes as `@private` is not enough to hide them
from showing up in the API doc at http://emberjs.com/api/data/.

This commit changes the used comments to start with `/*`, which are
definitely ignored by YUIDoc.
@pangratz
Copy link
Member Author

pangratz commented Jan 9, 2016

@bmac done

bmac added a commit that referenced this pull request Jan 9, 2016
…-docs

[DOC beta] Hide private classes from YUIDoc
@bmac bmac merged commit e926b26 into emberjs:master Jan 9, 2016
@pangratz pangratz deleted the make-internal-model-private-in-docs branch January 12, 2016 11:22
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.

None yet

4 participants