AdrianBZG / InterMine-Data-Browser-Tool

InterMine Data Browser: a tool for exploring semi-homogeneous biological datasets
http://data-browser.apps.intermine.org/
Other
4 stars 38 forks source link

Consider adding formal api documentation comments #43

Closed yochannah closed 6 years ago

yochannah commented 6 years ago

The code is generally neat, tidy, and readable, but it might be nice to add some js api docs if possible too - it requires structured comments on functions.

See https://github.com/documentationjs/documentation/blob/master/docs/GETTING_STARTED.md for an example - it's ok to consider other documentation tools if they suit you better though.

AdrianBZG commented 6 years ago

@yochannah @rachellyne The code is now documented according to documentation.js, also a new gulp task to generate the documentation for the code in src/ and place it as HTML in the docs/ folder has been made available (gulp documentation). Also the documentation is available in GitHub pages at https://adrianbzg.github.io/InterMine-Data-Browser-Tool/. 👍

image

yochannah commented 6 years ago

This is great - thanks for adding it :) I am not at all worried about picking up this code in the future - it's so well documented it should hopefully be quite easy.

AdrianBZG commented 6 years ago

@yochannah Awesome!