Skip to content
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

Tune react/jsx-curly-brace-presence #48

Merged

Conversation

tetsuharuohzeki
Copy link
Contributor

  • For children, I think we enable this.
    but the current rule impl. is pretty buggy and noisey. It's better to disable it.
  • For props, in JSX syntax, props is not a string in many cases.
    And in plain JavaScript worlds, we don't have any static type checks,
    we need think about whether we should add a bracket or not by the type of props.
    It's pretty tired thing.
    For sorting style and avoid this tired thing, I prefer to write bracket always.

#41

* For _children_, I think we enable this.
  but the current rule impl. is pretty buggy and noisey. It's better to disable it.
* For _props_, inn JSX syntax, `props` is not a string in many cases.
  And in plain JavaScript worlds, we don't have any static type checks,
  we need think about whether we should add a bracket or not by the type of props.
  It's pretty tired thing.
  For sorting style and avoid this tired thing, I prefer to write bracket always.
@tetsuharuohzeki tetsuharuohzeki added A-config AREA: changes related to rules I-enhancement labels Mar 5, 2019
@tetsuharuohzeki tetsuharuohzeki merged commit 9fa7de0 into cats-oss:master Mar 5, 2019
@tetsuharuohzeki tetsuharuohzeki deleted the disalbe-jsx-curly-space branch March 5, 2019 06:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-config AREA: changes related to rules I-enhancement L-TypeScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant