Skip to content
This repository was archived by the owner on Jan 19, 2019. It is now read-only.

Fix: Set name to type JSXIdentifier and fix selfClosing (fixes #172) #175

Merged
merged 1 commit into from
Feb 26, 2017

Conversation

soda0289
Copy link
Member

JSXAttribute name property should be a node with type JSXIdentifier.
Self closing tags cannot check the parent node since its parent is
the outside tag. This is because we convert the JSXSelfClosingElement to a
JSXOpeningElement and do not change the parent. This commit converts
the node and then sets selfClosing instead of depending on the parent node.

…#172)

JSXAttribute name property should be a node with type JSXIdentifier.
Selfclosing tags cannot check the parent node since its parent is
 the outside tag. This is becuase we convert the JSXSelfClosingElement to a
JSXOpeningElement and do not change the parent. This commit converts
the node and then sets selfClosing instead of depending on the parent node.
@eslintbot
Copy link

LGTM

Copy link
Member

@JamesHenry JamesHenry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

@JamesHenry JamesHenry merged commit 701e2c5 into eslint:master Feb 26, 2017
@soda0289 soda0289 deleted the fix-jsx branch May 23, 2017 13:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants