Repository navigation
chore: 🔨 Update dependencies - axios - #606
Conversation
pwright
left a comment
There was a problem hiding this comment.
do we need reviews on dependency updates?
|
@pwright @fgiorgetti @c-kruse as Paul mentioned do we need approval for future dependencies updates? |
|
@pierpman I don't think we've came up with a codified review requirements process - especially in the console project. In my opinion, one human's judgement is sufficient for dependency bumps (e.g. were dependabot involved, one approval -> merge; otherwise one of us manually upgrades -> self reviews -> merge if confident), especially for minor version bumps. TBH coderabbit would probably be more diligent about understanding the detla between axios 1.16 and 1.20 and deciding if it is relevant compared to me. |
|
That makes sense to me. I agree that one human approval should be sufficient for dependency updates, and also CodeRabbit would be a useful addition to the repository. I was trying to find the best way to update the dependencies. There are quite a few that need updating, so I might group related updates together to avoid creating too many PRs. |
Description
This PR updates the
axiosdependency from version1.16.1to1.20.0.Additional context (optional)
This is a minor version bump for the
axiosHTTP client library. The update from1.16.1to1.20.0may include bug fixes, security patches, and new features introduced across the intermediate releases. It is recommended to review the axios changelog for a full list of changes between these versions.A new commit has been added to this PR.
How to test
axiosis installed:axios@1.20.0.yarn test