This repository has been archived by the owner. It is now read-only.
Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upRemove jsdox #10254
Closed
Remove jsdox #10254
Comments
Merged
dfperry5
added a commit
to dfperry5/browser-laptop
that referenced
this issue
Aug 18, 2017
Fix brave#10254
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
There isn't much value in jsdox for the actions right now since someone can just open up the file and see the code and comments. It's annoying because when you commit it does the doc generation after.
It's also not working right now due to some problem with jsdox with maxBuffer execFile. For some reason the full stdout is being exceeded and maxBuffer option doesn't seem to be working.