chore: use semver range for axios - #190
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ChangesDependency Version Relaxation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Pinning the version of your dependencies, if you are the library provider, is not the best solution to security-related issues. On the contrary, it makes it more difficult to resolve problems when your dependencies encounter issues.
For example, the current version of
axioshas an issue: GHSA-q8qp-cvcw-x6jj:In my project, I would simply update
axiosand the problem would be sorted. But because the version is explicitly specified, I’ll have to useoverridesAdditionally, this creates a problem with duplicate dependencies, as the version of
axiosrequired by your package will be installed separately, even if the project specifies^1.15.0. This means that if the project usesaxiosversion^1.16.0, it will encounter issues when building the TypeScript projectSummary by CodeRabbit