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.

Sign up for GitHub

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

Jump to bottom

Add proper javadoc comments to core/lib/pattern_exporter.js #487

Closed
bmuenzenmeyer opened this issue Sep 30, 2016 · 1 comment
Closed

Add proper javadoc comments to core/lib/pattern_exporter.js #487

bmuenzenmeyer opened this issue Sep 30, 2016 · 1 comment
Labels
fix / refactor 🚧 staged for next release 🏁

Comments

@bmuenzenmeyer
Copy link
Member

Code is read more than it is written.

Akin to work already done in core/lib/ui_builder.js

Read the code, and describe each function and parameters accordingly.

https://en.wikipedia.org/wiki/JSDoc
@yarwest
Copy link

yarwest commented Sep 30, 2016

@bmuenzenmeyer Hey! I'd love to document this file for you :) I have experience working with JS and JSDoc.

spezzino added a commit to spezzino/patternlab-node that referenced this issue Oct 1, 2016
@spezzino
pattern-lab#487 Add proper javadoc comments to core/lib/pattern_expor…
94d5ad7 
…ter.js
spezzino added a commit to spezzino/patternlab-node that referenced this issue Oct 1, 2016
@spezzino
pattern-lab#487 Add proper javadoc comments to core/lib/pattern_expor…
871d93c 
…ter.js
bmuenzenmeyer added a commit that referenced this issue Oct 5, 2016
@bmuenzenmeyer
Merge pull request #495 from spezzino/issue/487
2fd4ca8 
#487 Add proper javadoc comments to core/lib/pattern_exporter.js
bmuenzenmeyer added a commit that referenced this issue Oct 5, 2016
@bmuenzenmeyer
add clarifying notes for #487
da064bf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fix / refactor 🚧 staged for next release 🏁
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bmuenzenmeyer @yarwest