Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Chore(learn): add CJS/ESM #3205

Closed
wants to merge 11 commits into from
Closed

Chore(learn): add CJS/ESM #3205

wants to merge 11 commits into from

Conversation

AugustinMauroy
Copy link
Contributor

Description

Adding a learn page about commonJS and Ecma Modules.

Related Issues

No related issues.

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

@codecov-commenter
Copy link

codecov-commenter commented Feb 5, 2023

Codecov Report

Base: 66.02% // Head: 66.25% // Increases project coverage by +0.23% 🎉

Coverage data is based on head (349d14a) compared to base (fd57b87).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3205      +/-   ##
==========================================
+ Coverage   66.02%   66.25%   +0.23%     
==========================================
  Files         118      145      +27     
  Lines        1351     1716     +365     
  Branches      342      398      +56     
==========================================
+ Hits          892     1137     +245     
- Misses        422      534     +112     
- Partials       37       45       +8     
Impacted Files Coverage Δ
src/templates/blog.tsx 0.00% <0.00%> (-100.00%) ⬇️
src/hooks/useDetectOs.ts 80.00% <0.00%> (-20.00%) ⬇️
util-node/getNodeReleasesData.js 91.30% <0.00%> (-6.32%) ⬇️
src/__fixtures__/page.tsx 94.11% <0.00%> (-5.89%) ⬇️
src/pages/index.tsx 100.00% <0.00%> (ø)
src/templates/api.tsx 0.00% <0.00%> (ø)
src/templates/learn.tsx 100.00% <0.00%> (ø)
util-node/createSlug.js 100.00% <0.00%> (ø)
util-node/getNvmData.js 100.00% <0.00%> (ø)
src/hooks/useOnScreen.tsx 100.00% <0.00%> (ø)
... and 187 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label Feb 7, 2023
@benhalverson benhalverson temporarily deployed to firebase-staging February 7, 2023 02:02 — with GitHub Actions Inactive
@github-actions github-actions bot removed the create-preview Generate preview on staging.nodejs.dev label Feb 7, 2023
@github-actions
Copy link

github-actions bot commented Feb 7, 2023

Please find a preview at: https://staging.nodejs.dev/3205/

content/learn/getting-started/esm-cjs.en.md Outdated Show resolved Hide resolved
content/learn/getting-started/esm-cjs.en.md Outdated Show resolved Hide resolved
content/learn/getting-started/esm-cjs.en.md Outdated Show resolved Hide resolved
content/learn/getting-started/esm-cjs.en.md Outdated Show resolved Hide resolved
content/learn/getting-started/esm-cjs.en.md Outdated Show resolved Hide resolved
AugustinMauroy and others added 5 commits February 7, 2023 17:52
Co-authored-by: Shanmughapriyan S <priyanshan03@gmail.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Shanmughapriyan S <priyanshan03@gmail.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Shanmughapriyan S <priyanshan03@gmail.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Shanmughapriyan S <priyanshan03@gmail.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
Co-authored-by: Shanmughapriyan S <priyanshan03@gmail.com>
Signed-off-by: Augustin Mauroy <97875033+AugustinMauroy@users.noreply.github.com>
@shanpriyan shanpriyan added the create-preview Generate preview on staging.nodejs.dev label Feb 7, 2023
@github-actions github-actions bot removed the create-preview Generate preview on staging.nodejs.dev label Feb 7, 2023
@github-actions
Copy link

github-actions bot commented Feb 7, 2023

Please find a preview at: https://staging.nodejs.dev/3205/

@shanpriyan shanpriyan requested review from ovflowd and removed request for ovflowd March 10, 2023 20:26
@ovflowd ovflowd closed this Mar 20, 2023
@AugustinMauroy
Copy link
Contributor Author

Why did you close it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants