chore: move modules to dedicated files - #3890
Conversation
✅ Deploy Preview for fakerjs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## next #3890 +/- ##
=======================================
Coverage 98.85% 98.85%
=======================================
Files 923 923
Lines 3224 3224
Branches 591 591
=======================================
Hits 3187 3187
Misses 33 33
Partials 4 4
🚀 New features to boost your workflow:
|
ST-DDT
left a comment
There was a problem hiding this comment.
This PR is very easy to review, because the files were just renamed and index reexports it.
While this is true, it does cause a lot of more work for the SMF PRs than just doing it as part of the SMF PRs.
I'll find a way to automate the SMF refactoring again and then just rebase my SMF PRs, so I don't loose all my work/don't have to reject this.
|
Looks like this needs a rebase. |
|
converted to draft, I wont have time before next monday |
8718bdf to
e8a45ed
Compare
|
This needs to be updated after that PR is merged: |
e8a45ed to
fb6e939
Compare
xDivisionByZerox
left a comment
There was a problem hiding this comment.
Can confirm that the implementation that was previously in **/index.ts now lives inside **/module.ts.

Uh oh!
There was an error while loading. Please reload this page.