Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(rdmd to rmdx): wrap scripts + styles in an <HTMLBlock> #954

Merged

Conversation

rafegoldberg
Copy link
Contributor

@rafegoldberg rafegoldberg commented Aug 27, 2024

🧰 Changes

  • wrap scripts + styles in an
  • add test cases
  • implement compatabilityMode option
    discussed with @kellyjosephprice and we're gonna deprecate compat mode for RMDX (8ad57ac)

🧬 QA & Testing

Pull this branch down and try running the test suite for the RDMD→MDX conversion:

npm run test -- compatability.test

If these new tests pass, this worked! 🎉

@rafegoldberg rafegoldberg force-pushed the chore/rdmd-to-mdx-conversion/wrap-script+style-tags branch from bf97d12 to cf2e306 Compare August 27, 2024 17:15
@rafegoldberg rafegoldberg force-pushed the chore/rdmd-to-mdx-conversion/wrap-script+style-tags branch from cf2e306 to da79e25 Compare August 27, 2024 17:17
Copy link
Collaborator

@kellyjosephprice kellyjosephprice left a comment

Choose a reason for hiding this comment

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

Tests look great

compatability mode has been deprecated for RMDX so raw `<script>` tags should never be executed
@rafegoldberg rafegoldberg marked this pull request as ready for review September 9, 2024 13:17
@rafegoldberg rafegoldberg force-pushed the chore/rdmd-to-mdx-conversion/wrap-script+style-tags branch from 37e77fa to 12e7e66 Compare September 9, 2024 13:19
@rafegoldberg rafegoldberg merged commit 2c878b6 into next Sep 9, 2024
13 checks passed
@rafegoldberg rafegoldberg deleted the chore/rdmd-to-mdx-conversion/wrap-script+style-tags branch September 9, 2024 14:38
rafegoldberg pushed a commit that referenced this pull request Sep 10, 2024
## Version 7.5.0

### ✨ New & Improved

* pass through stringify options ([#963](#963)) ([fb4fb7d](fb4fb7d))

### 🛠 Fixes & Updates

* **rdmd to rmdx:** wrap scripts + styles in an <HTMLBlock> ([#954](#954)) ([2c878b6](2c878b6))

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

This PR was released!

🚀 Changes included in v7.5.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.

2 participants