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

yuidoc . --theme simple doesn't work #51

Closed
LarryBattle opened this issue Jul 13, 2012 · 2 comments
Closed

yuidoc . --theme simple doesn't work #51

LarryBattle opened this issue Jul 13, 2012 · 2 comments
Assignees
Labels

Comments

@LarryBattle
Copy link

I'm trying to compile with the "simple" theme.
But when I type in yuidoc . -T simple, I get this error.

error: -------------------------------------------------------------------------
error: An uncaught YUIDoc error has occurred, stack trace given below
error: --------------------------------------------------------------------------
error:
ReferenceError: path is not defined
    at YUI.add.Y.Options (C:\Users\d\AppData\Roaming\npm\node_modules\yuidocjs
\lib\options.js:113:40)
    at Object.<anonymous> (C:\Users\d\AppData\Roaming\npm\node_modules\yuidocj
s\lib\cli.js:17:17)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.runMain (module.js:487:10)
    at process.startup.processNextTick.process._tickCallback (node.js:244:9)
error: -------------------------------------------------------------------------
-
error: Node.js version: v0.8.1
error: YUI version: 3.6.0pr3
error: YUIDoc version: 0.3.16
error: Please file all tickets here: http://github.com/yui/yuidoc/issues
error: --------------------------------------------------------------------------
@davglass davglass reopened this Jul 13, 2012
@ghost ghost assigned davglass Jul 13, 2012
@davglass
Copy link
Member

Technically the simple theme is not maintained, there has been much more work done on the default theme but I'll look into this anyway.

@LarryBattle
Copy link
Author

I fixed it. All you have to do is pass in the path variable to Y.Options from cli.js. I'm submitting a pull request.

davglass added a commit that referenced this issue Jul 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants