Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Remove jsdox #10254

Closed
bbondy opened this issue Aug 3, 2017 · 0 comments
Closed

Remove jsdox #10254

bbondy opened this issue Aug 3, 2017 · 0 comments
Assignees
@bbondy
Labels
dev-setup QA/no-qa-needed release-notes/exclude
Milestone
0.18.23 Hotfix

Comments

@bbondy
Copy link
Member

bbondy commented Aug 3, 2017

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.
@bbondy bbondy added this to the 0.18.x Hotfix milestone Aug 3, 2017
@bbondy bbondy self-assigned this Aug 3, 2017
bbondy added a commit that referenced this issue Aug 3, 2017
@bbondy
Remove jsdox
d0da35e 
Fix #10254
@bbondy bbondy mentioned this issue Aug 3, 2017
Merged
8 tasks
NejcZdovc pushed a commit that referenced this issue Aug 3, 2017
@bbondy @NejcZdovc
Remove jsdox
4eaa002 
Fix #10254
NejcZdovc pushed a commit that referenced this issue Aug 3, 2017
@bbondy @NejcZdovc
Remove jsdox
a36c9ef 
Fix #10254
NejcZdovc pushed a commit that referenced this issue Aug 3, 2017
@bbondy @NejcZdovc
Remove jsdox
2ab86b1 
Fix #10254
@alexwykoff alexwykoff mentioned this issue Aug 7, 2017
Closed
14 tasks
dfperry5 pushed a commit to dfperry5/browser-laptop that referenced this issue Aug 18, 2017
@bbondy @dfperry5
Remove jsdox
1ac9070 
Fix brave#10254
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bbondy bbondy

Labels
dev-setup QA/no-qa-needed release-notes/exclude
Projects
None yet
Milestone
0.18.23 Hotfix
Development

No branches or pull requests
2 participants
@bbondy @luixxiul