Skip to content

Support JSXElement. (no parse)#3

Merged
h13i32maru merged 1 commit into
esdoc:masterfrom
koba04:support-jsxelement
May 19, 2015
Merged

Support JSXElement. (no parse)#3
h13i32maru merged 1 commit into
esdoc:masterfrom
koba04:support-jsxelement

Conversation

@koba04

@koba04 koba04 commented May 19, 2015

Copy link
Copy Markdown
Contributor

Currently, an error occurs when I target files including jsx syntax like below.

import React from 'react';

class App extends React.Component {
  render() {
    return <div>Hello</div>;
  }
}
  • error trace
Error: Unknown node type JSXElement.
    at Controller.traverse (/Users/koba04/project/node_modules/esdoc/node_modules/estraverse/estraverse.js:515:31)
    at Object.traverse (/Users/koba04/project/node_modules/esdoc/node_modules/estraverse/estraverse.js:707:27)
    at Function.findPathInImportDeclaration (/Users/koba04/project/node_modules/esdoc/out/src/Util/ASTUtil.js:60:31)
    at ClassDoc._resolveLongname (/Users/koba04/project/node_modules/esdoc/out/src/Doc/AbstractDoc.js:943:51)
    at ClassDoc._extends (/Users/koba04/project/node_modules/esdoc/out/src/Doc/ClassDoc.js:137:29)
    at ClassDoc._apply (/Users/koba04/project/node_modules/esdoc/out/src/Doc/ClassDoc.js:53:23)
    at ClassDoc.AbstractDoc (/Users/koba04/project/node_modules/esdoc/out/src/Doc/AbstractDoc.js:54:10)
    at new ClassDoc (/Users/koba04/project/node_modules/esdoc/out/src/Doc/ClassDoc.js:36:20)
    at DocFactory._createDoc (/Users/koba04/project/node_modules/esdoc/out/src/Factory/DocFactory.js:320:14)
    at DocFactory._traverseComments (/Users/koba04/project/node_modules/esdoc/out/src/Factory/DocFactory.js:250:24)

This pull request is for fixing it.
This doesn't include parsing JSX tree.

h13i32maru added a commit that referenced this pull request May 19, 2015
Support JSXElement. (no parse)
@h13i32maru
h13i32maru merged commit ff55c2b into esdoc:master May 19, 2015
@h13i32maru

Copy link
Copy Markdown
Member

@koba04 Thank you this PR 😄

@koba04

koba04 commented May 20, 2015

Copy link
Copy Markdown
Contributor Author

thank you! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants