-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: missing code to wire-up file loader #145
Conversation
🦋 Changeset detectedLatest commit: 0ec28c1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov Report
@@ Coverage Diff @@
## master #145 +/- ##
==========================================
- Coverage 90.92% 90.77% -0.15%
==========================================
Files 78 79 +1
Lines 3130 3155 +25
==========================================
+ Hits 2846 2864 +18
- Misses 284 291 +7
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Some tests failed for |
Working on it but I don't understand what happened. Seems like unrelated tests start failing 🙄 |
…file-loader-missing-tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for improving the readability! One very little last thing: I will suggest renaming absolut
into absolute
, then I will approve the PR.
Co-authored-by: Ludovic Muller <ludovicm67@users.noreply.github.com>
Co-authored-by: Ludovic Muller <ludovicm67@users.noreply.github.com>
In #142 I forgot to add code which actually sets up the new loader in registry
Here, I did that and added an e2e test which uses the loader