-
Notifications
You must be signed in to change notification settings - Fork 131
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
javascript domain: bug on windows build #125
Comments
Yeah, perhaps a config option is the most correct fix here? I'm not familiar with windows development with node, but it seems like this shouldn't be magic that we need to use. Rather, is there something we're not doing with the subprocess call that would imply a |
I have been checking the subprocess doc and nothing looks like that. Safest to check the os in our code I guess. Optionnally configuring paths for required tools would be a nice bonus. |
Issue readthedocs#125. jsdoc on windows is deployed as jsdoc.cmd
lint + import os
Issue #125. jsdoc on windows is deployed as jsdoc.cmd
In mappers/javascript.py
On windows, the jsdoc command is actually named jsdoc.cmd and is not recognized by subprocess().
Proposed:
Could be nice to support an optional path for these commands in conf.py
The text was updated successfully, but these errors were encountered: