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

Missing theme after fresh installation #146

Closed
Laplandia opened this issue Sep 28, 2015 · 11 comments · Fixed by #212
Closed

Missing theme after fresh installation #146

Laplandia opened this issue Sep 28, 2015 · 11 comments · Fixed by #212

Comments

@Laplandia
Copy link

/usr/local/lib/node_modules/documentation/streams/output/lib/resolve_theme.js:17
throw new Error('Theme ' + theme + ' not found');
^
Error: Theme documentation-theme-default not found
at resolveTheme (/usr/local/lib/node_modules/documentation/streams/output/lib/resolve_theme.js:17:11)
at module.exports (/usr/local/lib/node_modules/documentation/streams/output/html.js:43:21)

@tmcw
Copy link
Member

tmcw commented Oct 30, 2015

@Laplandia can you try installing now and see if this issue remains with v3.0.0?

@luke-j
Copy link

luke-j commented Oct 31, 2015

This issue is still occurring on v3.0.0 for me.
Throwing the same error Theme documentation-theme-default not found.

@tmcw
Copy link
Member

tmcw commented Oct 31, 2015

Bizarre: this happens after npm install -g documentation, and you're calling just documentation on the cli?

@luke-j
Copy link

luke-j commented Oct 31, 2015

Yep, installed globally. I'm calling it with these options -o /tmp/ -p true -f html

@tmcw
Copy link
Member

tmcw commented Oct 31, 2015

Any other clues to help debug - are you using OSX or Linux? npm version? node version & installation method?

@luke-j
Copy link

luke-j commented Oct 31, 2015

I'm on a vagrant machine running:

  • Ubuntu v14.04.3 LTS
  • node v0.10.25
  • npm v1.3.10

The full error is:

/usr/local/lib/node_modules/documentation/bin/documentation.js:21
    throw err;
          ^
Error: Theme documentation-theme-default not found
    at resolveTheme (/usr/local/lib/node_modules/documentation/lib/resolve_theme.js:17:11)
    at makeHTML (/usr/local/lib/node_modules/documentation/lib/output/html.js:57:21)
    at /usr/local/lib/node_modules/documentation/bin/documentation.js:34:3
    at /usr/local/lib/node_modules/documentation/index.js:85:7
    at /usr/local/lib/node_modules/documentation/lib/input/dependency.js:41:5
    at ConcatStream.<anonymous> (/usr/local/lib/node_modules/documentation/node_modules/concat-stream/index.js:36:43)
    at ConcatStream.EventEmitter.emit (events.js:117:20)
    at finishMaybe (/usr/local/lib/node_modules/documentation/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:509:14)
    at endWritable (/usr/local/lib/node_modules/documentation/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:519:3)
    at ConcatStream.Writable.end (/usr/local/lib/node_modules/documentation/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:484:5)

Things that might be relevant:

  • JS written as ES6, transpired with babel and concatenated with browserify

@knownasilya
Copy link
Contributor

Seeing the same issue, just installed as a dev dep and using in npm scripts. I'm on OSX using Node 4.2.1 and using this command: documentation -f html -o docs lib/index.js

@knownasilya
Copy link
Contributor

So using __dirname instead of process.cwd() finds it, but probably wont from the user's node_modules..

@luke-j
Copy link

luke-j commented Nov 2, 2015

Just an FYI that after a fresh install I'm still getting the same error message.

@knownasilya
Copy link
Contributor

@luke-j I don't think a new version has yet been released.

@tmcw
Copy link
Member

tmcw commented Nov 2, 2015

Just released 3.0.1, a version that includes this fix.

@tmcw tmcw reopened this Nov 2, 2015
@tmcw tmcw closed this as completed Nov 2, 2015
rhendric pushed a commit to rhendric/documentation that referenced this issue Sep 15, 2022
* basic error info for RedefinedIdent

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

Successfully merging a pull request may close this issue.

4 participants