Skip to content

fix(typescript): support module Node16/NodeNext#1005

Merged
Shinigami92 merged 1 commit intofaker-js:mainfrom
julienfouilhe:nodenext
May 25, 2022
Merged

fix(typescript): support module Node16/NodeNext#1005
Shinigami92 merged 1 commit intofaker-js:mainfrom
julienfouilhe:nodenext

Conversation

@julienfouilhe
Copy link
Copy Markdown
Contributor

Fix #1004

Recreating #979 (without changing the base "types" field)

@julienfouilhe julienfouilhe requested a review from a team as a code owner May 25, 2022 17:21
@Shinigami92 Shinigami92 added c: bug Something isn't working p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug labels May 25, 2022
@Shinigami92 Shinigami92 added this to the v7 - Current Major milestone May 25, 2022
@Shinigami92
Copy link
Copy Markdown
Member

After this is merged I would like to release the next version of faker

@Shinigami92 Shinigami92 requested review from a team May 25, 2022 17:22
@Shinigami92 Shinigami92 changed the title fix(typescript): Add support for new typescript "module": "node16" with node and ESM fix(typescript): support module Node16/NodeNext May 25, 2022
@Shinigami92 Shinigami92 enabled auto-merge (squash) May 25, 2022 17:25
@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2022

Codecov Report

Merging #1005 (99487db) into main (ccd2959) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1005      +/-   ##
==========================================
- Coverage   99.67%   99.67%   -0.01%     
==========================================
  Files        2113     2113              
  Lines      227001   227001              
  Branches      983      982       -1     
==========================================
- Hits       226256   226253       -3     
- Misses        725      728       +3     
  Partials       20       20              
Impacted Files Coverage Δ
src/modules/finance/index.ts 99.31% <0.00%> (-0.69%) ⬇️

@Shinigami92 Shinigami92 merged commit 8736c2a into faker-js:main May 25, 2022
@julienfouilhe julienfouilhe deleted the nodenext branch May 25, 2022 17:31
Minozzzi pushed a commit to Minozzzi/faker that referenced this pull request Jul 19, 2022
Co-authored-by: Andrew Ross <rossirsa@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: bug Something isn't working p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Types are not found when using node with ESM and typescript@4.7.2 with NodeNext

3 participants