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

Review JSDoc comments @fabianmoronzirfas #175

Open
2 of 3 tasks
ff6347 opened this issue Feb 21, 2017 · 3 comments
Open
2 of 3 tasks

Review JSDoc comments @fabianmoronzirfas #175

ff6347 opened this issue Feb 21, 2017 · 3 comments
Assignees

Comments

@ff6347
Copy link
Member

ff6347 commented Feb 21, 2017

  • structure.js
  • transformation.js
  • typography.js
@ff6347 ff6347 self-assigned this Feb 21, 2017
@ff6347 ff6347 mentioned this issue Feb 25, 2017
@ff6347
Copy link
Member Author

ff6347 commented Apr 11, 2017

basil.js/data.js at master · basiljs/basil.js
https://github.com/basiljs/basil.js/blob/master/src/includes/data.js#L232

Is this line needed?

This more or less a reminder for myself. Is this jsDoc comment needed?

@b-g
Copy link
Member

b-g commented Apr 11, 2017

I don't think so ... most likely a hack to have the old docs look a certain way or a mistake.

@ff6347
Copy link
Member Author

ff6347 commented Apr 11, 2017

Oki. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants