Skip to content

Conversation

@kellyjosephprice
Copy link
Collaborator

@kellyjosephprice kellyjosephprice commented Dec 11, 2024

PR App Ref CX-1610

🧰 Changes

Fixes migrating tables with leading escapes.

The phrasing util is not aware of the escape node type, as it is from a previous version of mdast. This was confusing our tables-to-jsx transformer. It would look at tables and decide if it could be compiled to markdown format, eg:

| a | b |
|---|---|
| 1 | 2 |

or if it should be compiled to JSX format, eg:

<Table>
  <TableHead>
    <TableRow>
      <TableCell>a</TableCell>
      <TableCell>b</TableCell>
    </TableRow>
  </TableHead>
  <TableBody>
    <TableRow>
      <TableCell>1</TableCell>
      <TableCell>2</TableCell>
    </TableRow>
  </TableBody>
</Table>

🧬 QA & Testing

@kellyjosephprice kellyjosephprice marked this pull request as ready for review December 11, 2024 23:38
@kellyjosephprice kellyjosephprice merged commit 4692ece into next Dec 11, 2024
13 checks passed
@kellyjosephprice kellyjosephprice deleted the fix/table-escapes branch December 11, 2024 23:48
rafegoldberg pushed a commit that referenced this pull request Jan 8, 2025
## Version 7.11.0
### ✨ New & Improved

* add title props to plain ([#1036](#1036)) ([a3b7f50](a3b7f50))

### 🛠 Fixes & Updates

* **deps:** bump debug from 4.3.7 to 4.4.0 ([#1038](#1038)) ([a5a8061](a5a8061))
* **deps:** bump readmeio/rdme from 8 to 9 ([#1037](#1037)) ([3f403de](3f403de)), closes [/github.com/readmeio/rdme/blob/HEAD/#migrating-to-rdme10](https://github.com/readmeio//github.com/readmeio/rdme/blob/HEAD//issues/migrating-to-rdme10) [#1099](#1099) [#1085](#1085) [#1106](#1106) [#1107](#1107) [#1068](#1068) [#962](#962) [#1067](#1067) [github.com/readmeio/rdme/blob/d01d76fe3c2e4a98b4f5c415be03e589fbe3b56e/.releaserc.yml#L30](https://github.com/readmeio/github.com/readmeio/rdme/blob/d01d76fe3c2e4a98b4f5c415be03e589fbe3b56e/.releaserc.yml/issues/L30) [#1067](#1067) [#1072](#1072) [#1100](#1100) [#1078](#1078) [#1042](#1042) [#908](#908) [#901](#901) [#1073](#1073) [#1080](#1080) [#961](#961) [#963](#963) [#993](#993) [github.com/nodejs/node/blob/main/doc/changelogs/CHANGELOG_V20.md#2023-11-22-version-20100](https://github.com/readmeio/github.com/nodejs/node/blob/main/doc/changelogs/CHANGELOG_V20.md/issues/2023-11-22-version-20100) [#965](#965) [#1080](#1080) [#1106](#1106) [#1105](#1105) [#1098](#1098) [#1099](#1099) [#1100](#1100)
* dont treat escapes as flow ([#1040](#1040)) ([4692ece](4692ece))

<!--SKIP CI-->
@rafegoldberg
Copy link
Contributor

This PR was released!

🚀 Changes included in v7.11.0

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants