Skip to content

Doc coverage/quality reporting #284

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

Closed
seaneagan opened this issue Mar 19, 2015 · 5 comments
Closed

Doc coverage/quality reporting #284

seaneagan opened this issue Mar 19, 2015 · 5 comments
Labels
contributions-welcome Contributions welcome to help resolve this (the resolution is expected to be clear from the issue) P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug

Comments

@seaneagan
Copy link
Contributor

'twould be awesome to have detailed reporting on doc coverage/quality ala inchjs.

Reasons to build this into dartdoc rather than as a separate tool:

  • Code reuse.
  • Speed. Doc generation and quality gathering can be done simultaneously similar to how test coverage and test results can be calculated from a single run of the unit tests.
  • easier integration into dartdocs.org

dartdocs.org could provide a url to access the doc quality page, and also a badge that people could include in their READMEs, and maybe directly integrate it into pub.dartlang.org as well.

Another part would be linter support so that editors can flag and link to undocumented (or poorly documented) things in the source to make them easy to fix.

@sethladd
Copy link
Contributor

Indeed! We have something like that here: http://google.github.io/dart-shapeshift/

We would LOVE if someone integrated it with the new dartdoc.

Patches welcome

@pq
Copy link
Member

pq commented Mar 19, 2015

👍

I've been thinking about this vis a vis linting. For my edification, how do we specify public API? Are we limiting ourselves to everything in lib/ (minus lib/src)? Do we look at bin?

@keertip
Copy link
Collaborator

keertip commented Mar 20, 2015

We don't look at bin, just lib/ (minus lib/src).

@pq
Copy link
Member

pq commented Mar 24, 2015

Actually, it's more nuanced I think. It looks like you also process explicitly exported members.

Look at the docs for barback for example:

http://www.dartdocs.org/documentation/barback/0.15.2+4/index.html#barback/barback

And now notice that much of that is in fact exported from lib/src:

https://github.com/dart-lang/barback/blob/master/lib/barback.dart

(Thanks to @munificent for the salient example!)

@sethladd sethladd added type-enhancement A request for a change that isn't a bug contributions-welcome Contributions welcome to help resolve this (the resolution is expected to be clear from the issue) P2 A bug or feature request we're likely to work on labels Apr 2, 2015
@sethladd
Copy link
Contributor

This would be cool, but we're unlikely to get to this. If anyone is interested, please reopen and take a shot at it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributions-welcome Contributions welcome to help resolve this (the resolution is expected to be clear from the issue) P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

4 participants