Adds `replaceJsxBody()` support for when parent node is root #2085

cartogram · 2021-11-23T16:41:32Z

Description

The replaceJsxBody transform would fail if it was operating at the top (root) of the react tree. This PR fixes this and adds a test.

Patch: Bug (non-breaking change which fixes an issue)
Minor: New feature (non-breaking change which adds functionality)
Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

packages/ast-utilities/CHANGELOG.md

add root node support to replace jsx node

a60ec9b

cartogram requested review from a team and jplhomer November 23, 2021 16:41

cartogram commented Nov 23, 2021

View reviewed changes

packages/ast-utilities/CHANGELOG.md Outdated Show resolved Hide resolved

Update packages/ast-utilities/CHANGELOG.md

116c879

jplhomer approved these changes Nov 23, 2021

View reviewed changes

cartogram mentioned this pull request Nov 23, 2021

New transform wrapJsxChildren() #2086

Merged

4 tasks

cartogram merged commit 37d1d0c into main Nov 24, 2021

cartogram deleted the @cartogram/replace-jsx-root branch November 24, 2021 16:21