chore(content): use as 100% new codebox
Description
Adding possibility to switch between cjs/esm on new codeBox
Related Issues
No related Issues
Check List
- [X] I have read the Contributing Guidelines and made commit messages that follow the guideline.
- [X] I have run
npx turbo formatto ensure the code follows the style guide. - [X] I have run
npx turbo testto check if all tests are passing. - [X] I have run
npx turbo buildto check if the website builds without errors. - NA I've covered new added functionality with unit tests if necessary.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated (UTC) |
|---|---|---|---|
| nodejs-org | ✅ Ready (Inspect) | Visit Preview | Mar 4, 2024 3:15pm |
Can you attach a screenshot of an example?
Lighthouse Results
| URL | Performance | Accessibility | Best Practices | SEO | Report |
|---|---|---|---|---|---|
| /en | 🟢 100 | 🟢 96 | 🟢 100 | 🟢 91 | 🔗 |
| /en/about | 🟢 100 | 🟢 97 | 🟢 100 | 🟢 92 | 🔗 |
| /en/about/previous-releases | 🟢 98 | 🟢 95 | 🟢 100 | 🟢 92 | 🔗 |
| /en/download | 🟢 100 | 🟢 95 | 🟢 100 | 🟢 91 | 🔗 |
| /en/blog | 🟢 99 | 🟢 96 | 🟢 100 | 🟢 92 | 🔗 |
Unit Test Coverage Report
| Lines | Statements | Branches | Functions |
|---|---|---|---|
| 80.18% (433/540) | 79.65% (137/172) | 73.07% (76/104) |
Unit Test Report
| Tests | Skipped | Failures | Errors | Time |
|---|---|---|---|---|
| 88 | 0 :zzz: | 0 :x: | 0 :fire: | 4.561s :stopwatch: |
@ovflowd here you have a record. don't forget you have the preview ;)
https://github.com/nodejs/nodejs.org/assets/97875033/fe888e28-92ab-4fc3-91c4-942b634a97b6
I wouldn't mind the second tab being "ESM" instead - perhaps more recognisable?
I guess it's because on our docs we write "MJS", but I guess ESM would be more friendly for newcomers indeed