fix: tslib should be listed as dependency#1186
Conversation
Fixes MODULE_NOT_FOUND error when used in a non-ts project fixes prettier#1185
WalkthroughThe change moves the "tslib" package from the "devDependencies" section to the "dependencies" section in Changes
Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to 52ee63b in 33 seconds. Click for details.
- Reviewed
20lines of code in1files - Skipped
0files when reviewing. - Skipped posting
4draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:65
- Draft comment:
Correctly declaring tslib as a dependency ensures it’s available at runtime; this should resolve the MODULE_NOT_FOUND error. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. package.json:99
- Draft comment:
Removing tslib from devDependencies prevents conflicts and clarifies that it's required for production use. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
3. package.json:65
- Draft comment:
Good change: Moving 'tslib' to dependencies ensures it's installed for runtime, fixing the MODULE_NOT_FOUND error for non-TypeScript projects. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
4. package.json:101
- Draft comment:
Removing 'tslib' from devDependencies is appropriate since it's required at runtime rather than just during development. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
Workflow ID: wflow_OSkbY0B1m0IPhTUv
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1186 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 2 2
Lines 304 304
Branches 78 79 +1
=========================================
Hits 304 304 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
🦋 Changeset detectedLatest commit: fb3b3fe The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
tslib should be listed as dependency
Fixes MODULE_NOT_FOUND error when used in a non-ts project
I think this is all it takes, but I'm really not a javascript guy.
fixes #1185
Important
Move
tslibtodependenciesinpackage.jsonto fixMODULE_NOT_FOUNDerror in non-TypeScript projects.tslibfromdevDependenciestodependenciesinpackage.jsonto fixMODULE_NOT_FOUNDerror in non-TypeScript projects.This description was created by
for 52ee63b. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit