Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Enforce use of <>/</> shorthand instead of <React.Fragment>/</React.Fragment> #57

Closed
wincent opened this issue Aug 6, 2019 · 0 comments · Fixed by #58
Closed

Enforce use of <>/</> shorthand instead of <React.Fragment>/</React.Fragment> #57

wincent opened this issue Aug 6, 2019 · 0 comments · Fixed by #58
Assignees

Comments

@wincent
Copy link
Contributor

wincent commented Aug 6, 2019

We can get this via the "jsx-fragments" rule. See: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-fragments.md

We can apply this now to the liferay-portal master branch, because that is jsdoc 3.6.3 (older versions would choke on the shorthand).

On liferay-portal-ee master-private, we are still on the older jsdoc; will need to make sure that we update that there before applying this change (we have some React in there, and precisely 5 fragments at the time of writing).

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 a pull request may close this issue.

1 participant